[PATCH] cvsps: use commitids (when present) to detect changesets

Frank Kingswood frankk at kingswood-consulting.co.uk
Fri Jan 11 11:38:23 CST 2013


On 09/01/13 21:18, Bryan O'Sullivan wrote:

> Since the new output is different from the old output no matter which
> version of CVS I use, I suspect that your patch has introduced a regression.

Okay I have a patch which I thought I had sent yesterday but it seems it 
did not go to the list, apologies. That fixes a number of issues and 
does a bit of cleanup (for example that huge conditional).

I'll resend that patch this weekend but it is not actually correct 
either, but better. Someone who actually uses CVS should have a look. I 
think Martin wrote bits of that code, so I've CCed him in the hope that 
he might be the right person.

> I don't know squat about CVS, so I am not the right person to clean this
> up. Please run the test suite from the crew repo, figure out what the

So I'm hoping for help with this, I'm not able to spend more than 
trivial time on maintaining cvs conversion.

Frank


More information about the Mercurial-devel mailing list