Re: cvs2git reports a "sprout" from a nonexistent commit?

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Michael Haggerty <mhagger(at)alum(dot)mit(dot)edu>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Max Bowsher <maxb(at)f2s(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: cvs2git reports a "sprout" from a nonexistent commit?
Date: 2010-09-13 15:52:39
Message-ID: 13470.1284393159@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Michael Haggerty <mhagger(at)alum(dot)mit(dot)edu> writes:
> On 09/13/2010 03:14 AM, Tom Lane wrote:
>> Now as far as I can tell, the branch was made immediately before those
>> test commits you can see Marc making in each branch. In particular,
>> it was definitely made *after* Bryan deleted the src/bin/monitor files,
>> because neither of them have REL2_0 or REL2_0B tags. So why did cvs2git
>> choose to sprout the branch from the commit before that, and have to
>> duplicate the deletion of the files?

> This is a known bug 139 [1] in cvs2git's symbol parent choosing code
> (previously mentioned as part of bug 55 [2]). Patches are welcome :-)

Thanks for the response. Since this bug causes only one very minor
infelicity in our conversion, probably nobody here is going to be
motivated to fix it. OTOH, I'm still annoyed by the behavior for
addition of files to back branches ...

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Greg Smith 2010-09-13 16:05:30 Re: Perf regression in 2.6.32 (Ubuntu 10.04 LTS)
Previous Message Tom Lane 2010-09-13 15:48:22 Re: Report: removing the inconsistencies in our CVS->git conversion