Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Oct 2021 09:48:26 +0200
From:      Stefan Esser <se@freebsd.org>
To:        Jessica Clarke <jrtc27@freebsd.org>, Mark Johnston <markj@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: ded0d3d486eb - main - Add back Windows only VCXProj files
Message-ID:  <bb246831-724b-1526-fadf-40daee5fb2d7@freebsd.org>
In-Reply-To: <BD73C617-BCAD-4336-9EAA-5996889FDC8A@freebsd.org>
References:  <202110042015.194KF6pt032075@gitrepo.freebsd.org> <YVuH/avNrMyGSVg0@nuc> <BD73C617-BCAD-4336-9EAA-5996889FDC8A@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--------------07HR1lYFMLHAqGQ445BAcwWM
Content-Type: multipart/mixed; boundary="------------2HbEKl1G50neiEd9rJpdlGNN";
 protected-headers="v1"
From: Stefan Esser <se@freebsd.org>
To: Jessica Clarke <jrtc27@freebsd.org>, Mark Johnston <markj@freebsd.org>
Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org,
 dev-commits-src-main@freebsd.org
Message-ID: <bb246831-724b-1526-fadf-40daee5fb2d7@freebsd.org>
Subject: Re: git: ded0d3d486eb - main - Add back Windows only VCXProj files
References: <202110042015.194KF6pt032075@gitrepo.freebsd.org>
 <YVuH/avNrMyGSVg0@nuc> <BD73C617-BCAD-4336-9EAA-5996889FDC8A@freebsd.org>
In-Reply-To: <BD73C617-BCAD-4336-9EAA-5996889FDC8A@freebsd.org>

--------------2HbEKl1G50neiEd9rJpdlGNN
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Am 05.10.21 um 01:10 schrieb Jessica Clarke:
> On 5 Oct 2021, at 00:02, Mark Johnston <markj@freebsd.org> wrote:
>>
>> On Mon, Oct 04, 2021 at 08:15:06PM +0000, Stefan E=C3=9Fer wrote:
>>> The branch main has been updated by se:
>>>
>>> URL: https://cgit.FreeBSD.org/src/commit/?id=3Dded0d3d486eba766f61307=
0de0ed12029a9a3816
>>>
>>> commit ded0d3d486eba766f613070de0ed12029a9a3816
>>> Author:     Stefan E=C3=9Fer <se@FreeBSD.org>
>>> AuthorDate: 2021-07-28 13:35:16 +0000
>>> Commit:     Stefan E=C3=9Fer <se@FreeBSD.org>
>>> CommitDate: 2021-10-04 20:10:52 +0000
>>>
>>>    Add back Windows only VCXProj files
>>> ---
>>> bc.vcxproj          | 278 +++++++++++++++++++++++++++++++++++++++++++=
+++++++++
>>> bc.vcxproj.filters  | 182 ++++++++++++++++++++++++++++++++++
>>> bcl.vcxproj         | 161 ++++++++++++++++++++++++++++++
>>> bcl.vcxproj.filters |  96 ++++++++++++++++++
>>> 4 files changed, 717 insertions(+)
>>
>> Presumably these should instead have been added under contrib/bc?

No - they should not have been there. I did not notice that they
were put into the root of the source tree, I definitely did not
put them there. I performed a diff -r to verify the contents of
contrib/bc matched vendor/bc - but had not assumed or noticed that
there appeared files outside my work directory. (And I did not add
them to the commit. I used "git add -A ." in contrib/bc and had
not thought about the possibility that files outside that directory
could become part of the commit.)

I'm still fighting a lot with git and although I understand its
concepts and data structures often do not get the results I wanted
to achieve.

> I think they were meant to go into the root of the vendor branch, given=

> the last import in the branch also lost them, not just the subtree
> merge (and then presumably merged into contrib/bc).

I have no idea how this old issue resurfaced during this commit.

There was an issue with the CR-LF line separators used in these
files and .gitattributes missing in the initial commit, since I
had not been aware of the special handling of those files.

This has been corrected in the repository, but during this commit
a rebase was performed and somehow two old commits survived into
the commit of the MFV of bc-5.1.0 to contrib/bc (where these files
actually have been moved into a "vs" sub-directory with modified
contents).

It seems that git did resurrect those files not in contrib/bc but
in /usr/src for reasons that I do not understand. It seems that
git somehow replayed the old commit (see the AuthorDate from July)
but at a made up place ...

I'll delete these files and I hope they'll not show up in any further
commit I do in this area. I'll take more care to check the (long)
list of files that are part of the MFV.

Thanks for notifying me and sorry for messing up the repository
with that commit.

Regards, STefan

--------------2HbEKl1G50neiEd9rJpdlGNN--

--------------07HR1lYFMLHAqGQ445BAcwWM
Content-Type: application/pgp-signature; name="OpenPGP_signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="OpenPGP_signature"

-----BEGIN PGP SIGNATURE-----

wsB5BAABCAAjFiEEo3HqZZwL7MgrcVMTR+u171r99UQFAmFcA0oFAwAAAAAACgkQR+u171r99UTS
QAf/brri2p5Zx2eI5EasZTIY5iKFI0TDKfL8RF1siVG1XKPNqh/bWyU1KZqHQLVy/zAHWl171rwn
7RPQFiL5ZizNnYKlp+DykAm8G/iF7xSttyo/1cAdq+cZ5kJ+kc8c8jMUuOnUZ8scVMTfJSEfSWkS
ej48iOHmr4RUlFuitfqpNQKe2xXpk3cf1FdZLCRGsyCJTJW0IyDFzE6g68f0+eTMBsNfW6+rRo4w
U9nxWEjK6nWgoqf1oOTK13cU22qIN5vJXZue459tEgzb6CudRJ3zl1f6n1n100seqQMiaD92ZnSN
UxOy+AWNLiMFEIaB7b9c7ewfCbDjbMVqFcrvyZCZUQ==
=xoLJ
-----END PGP SIGNATURE-----

--------------07HR1lYFMLHAqGQ445BAcwWM--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bb246831-724b-1526-fadf-40daee5fb2d7>