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>

next in thread | previous in thread | raw e-mail | index | archive | help
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.n=
et> wrote:
>>=20
>>>> Author: ngie
>>>> Date: Wed Feb 13 05:41:04 2019
>>>> New Revision: 344087
>>>> URL: https://svnweb.freebsd.org/changeset/base/344087
>>>>=20
>>>> Log:
>>>> Compile the tests without -Werror for now
>>>>=20
>>>> I'm working on resolving the issue upstream on github
>>>>=20
>>>> Modified:
>>>> projects/import-googletest-1.8.1/lib/googletest/gmock/tests/Makefile
>>>> projects/import-googletest-1.8.1/lib/googletest/gtest/tests/Makefile
>>>=20
>>> 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?
>>=20
>> 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.
>=20
> 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.
>=20
> 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/stru=
cture was correct: ^/vendor/google/googletest/dist -> ^/vendor/google/google=
test/1.8.1 (for the release branch, given that the initial import was 1.8.1)=
.

I=E2=80=99ve been following similar conventions before in the past other pro=
jects I=E2=80=99ve imported. Given that this is the first of a handful of go=
ogle projects that will be imported, I don=E2=80=99t understand why there=E2=
=80=99s an issue with this naming convention.

-Enji=



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