Date: Wed, 4 Mar 2020 23:32:14 +0100 From: =?UTF-8?Q?Ulrich_Sp=C3=B6rlein?= <uqs@freebsd.org> To: Ed Maste <emaste@freebsd.org> Cc: freebsd-git@freebsd.org Subject: Re: Another svn-git oddity Message-ID: <CAJ9axoTo=YUwexpgGVuGWBvqL3ybanrQbrPEqpnRS1%2B31gVY-A@mail.gmail.com> In-Reply-To: <CAPyFy2ApMj3pD8FwGeA5ahrcZS0JUvuBD306TDQdSstJQk9mbQ@mail.gmail.com> References: <CAPyFy2A8Q=N%2BSS8=wOsfBtZ49QTUVXF2ZRYvUZX2dk=KyhKH-g@mail.gmail.com> <CAPyFy2DvaR5S6Vaz5JYKqyqGdPCaRg5Kyyn9aDb1EyXBqnA_Ug@mail.gmail.com> <CAPyFy2ApMj3pD8FwGeA5ahrcZS0JUvuBD306TDQdSstJQk9mbQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Am Mi., 4. M=C3=A4rz 2020 um 20:53 Uhr schrieb Ed Maste <emaste@freebsd.org= >: > On Wed, 4 Mar 2020 at 03:33, Ed Maste <emaste@freebsd.org> wrote: > > > > On Fri, 22 Nov 2019 at 08:52, Ed Maste <emaste@freebsd.org> wrote: > > > > > > As mentioned before there are some oddities in our git history - svn > > > commits translated to git in a slightly strange way. These have been > > > addressed and if/when we run the conversion from the beginning they'l= l > > > disappear. > > > > > > I've just found another oddity that affected attempts to use git > > > subtree for prototyping a method of handling vendor code in our tree. > > > I'm not sure yet if the changes to the converter would address this > > > case or not. > > > > > > The affected commits: > > > svn git Headline > > > r300828 00d578928e Create branch for bhyve graphics import. > > > r300829 b1cfc58d46 Initial bhyve native graphics support. > > > r302332 6775e12557 Import bhyve_graphics into CURRENT. Thanks to all > who > > > > These commits still exist in uqs's latest prototype conversion, with > hashes: > > 953b4a972bec > > 8efdfe5ee821 > > 6d7144acee3e > > r300828 branched (only) usr.sbin/bhyve/ to the top level directory, > which is represented in the git conversion as deleting everything > else. It's this that is the root cause of the git subtree trouble, I > believe. The actual changes come in r300829, merged in r302332. > > Uli, do you think we could just omit r300828 (and r300829) from the > svn2git conversion? This is a slight revision to the actual history, > but I think no information is really lost -- the merge in r302332 > includes the full commit message of the only change that was made on > the branch. > SGTM, I've added these to the exclusion list for the next incarnation. > > I do not believe there are other branches made in a similar way to > r300828 (copying just a single subdirectory) but am not certain and am > trying to verify that now. > Can you check this in programmatically across all branches? Cheers Uli
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ9axoTo=YUwexpgGVuGWBvqL3ybanrQbrPEqpnRS1%2B31gVY-A>