Re: git-cvsimport-3 and incremental imports
- From
Eric S. Raymond <esr@thyrsus.com>
- Date
- Jan 21, 2013, 12:43 UTC
- Message-ID
- <20130121124340.GA32219@thyrsus.com>
- In-Reply-To
- <20130121120010.GE7498@serenity.lan>
John Keeping <john@keeping.me.uk>:
> I also disagree that cvsps outputs commits *newer* than T since it will > also output commits *at* T, which is what I changed with the patch in my > previous message.
Ah. OK, that is yet another bug inherited from 2.x - the code doesn't match the documented (and correct) behavior. Please send me a patch against the cvsps repo, I'll merge it.
> Perhaps it is simplest to just save a CVS_LAST_IMPORT_TIME file in > $GIT_DIR and not worry about it any more.
Yes, I think you're right. Trying to carry that information in-band would probably doom us to all sorts of bug-prone complications.
Thanks for the good analysis. I wish everybody I had to chase bugs with could explain them with such clarity and concision.
Sigh. Now I have to figure out if cvsps's behavior can be rescued in Chris Rorvick's recently-discovered failure case. I'm not optimistic.
-- <a href="http://www.catb.org/~esr/">Eric S. Raymond</a>