Re: [PATCH] CVSps fixed; git-cvsimport works, too

From: Yann Dirson <ydirson@altern.org>
Date: Jun 17, 2006, 18:51 UTC
Message-ID: <20060617185126.GS7766@nowhere.earth>
In-Reply-To: <20060617143443.GA29602@pe.Belkin>

On Sat, Jun 17, 2006 at 10:34:43AM -0400, Chris Shoemaker wrote:

> Are you saying there was no pre-existing cache file when the patched
> version was first run?

It was the sample script you provided to demonstrate the problem which showed the new issue: it does a "cvsps -x" first, then runs git-cvsimport, which runs "cvsps -u". That one does fails. I'm pretty sure the cache does not exist beforehand, since your script creates the repo in a temporary directory.

BTW, I'll see about adding a testsuite to cvsps, based on the framework used by git. All these scriptlets used to exhibit the various problems have to be kept somewhere.

>  Did you delete it?  If so, then there's a bug somewhere.

I do think there is a bug somewhere :)

> The complaint is basically, Hey, there's an initial branch
> determination already made for this file, but it's not the one I would
> have made, so it must have been restored from a cache file that I
> didn't make.

OK, thanks for the clarification.

Best regards,

-- 
Yann Dirson    <ydirson@altern.org> |
Debian-related: <dirson@debian.org> |   Support Debian GNU/Linux:
                                    |  Freedom, Power, Stability, Gratis
     http://ydirson.free.fr/        | Check <http://www.debian.org/>