Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Feb 2019 07:08:58 -0800
From:      Enji Cooper <yaneurabeya@gmail.com>
To:        rgrimes@freebsd.org
Cc:        Enji Cooper <ngie@freebsd.org>, src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   Re: svn commit: r344087 - in projects/import-googletest-1.8.1/lib/googletest: gmock/tests gtest/tests
Message-ID:  <7F4DB347-CD49-4A09-A360-7DBAB33EEBC6@gmail.com>
In-Reply-To: <201902131442.x1DEgcsa081703@pdx.rh.CN85.dnsmgr.net>
References:  <201902131442.x1DEgcsa081703@pdx.rh.CN85.dnsmgr.net>

index | next in thread | previous in thread | raw e-mail

On Feb 13, 2019, at 06:42, Rodney W. Grimes <freebsd@pdx.rh.cn85.dnsmgr.net> wrote:

>> On Feb 13, 2019, at 06:26, Rodney W. Grimes <freebsd@pdx.rh.cn85.dnsmgr.net> wrote:
>> 
>>>> Author: ngie
>>>> Date: Wed Feb 13 05:41:04 2019
>>>> New Revision: 344087
>>>> URL: https://svnweb.freebsd.org/changeset/base/344087
>>>> 
>>>> Log:
>>>> Compile the tests without -Werror for now
>>>> 
>>>> I'm working on resolving the issue upstream on github
>>>> 
>>>> Modified:
>>>> projects/import-googletest-1.8.1/lib/googletest/gmock/tests/Makefile
>>>> projects/import-googletest-1.8.1/lib/googletest/gtest/tests/Makefile
>>> 
>>> Are we going to have to live with this project name of
>>> import-googletest-1.8.1 long term, or is this just a
>>> stepping stone?
>> 
>> It?s a short lived project. I didn?t realize in the last year and a
>> half that branch names had formalized into something else, especially
>> since GUIDELINES.txt doesn?t state a naming convention.
> 
> I do not know that they have formalized in any way.  But as long
> as it is shorted lived it really does not matter much.
> 
> Please do be careful when choosing a vendor/ directory name
> if that is where this is headed, 'import' and '1.8.1' are
> probably not wanted there at all, just googletest.

brooks reviewed my initial import into vendor to ensure that the naming/structure was correct: ^/vendor/google/googletest/dist -> ^/vendor/google/googletest/1.8.1 (for the release branch, given that the initial import was 1.8.1).

I’ve been following similar conventions before in the past other projects I’ve imported. Given that this is the first of a handful of google projects that will be imported, I don’t understand why there’s an issue with this naming convention.

-Enji

help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7F4DB347-CD49-4A09-A360-7DBAB33EEBC6>