[PATCH] Make test-convert-cvs work with older cvs versions

Patrick Mézard pmezard at gmail.com
Wed Oct 3 07:26:01 CDT 2007


Christian Ebert a écrit :
> * Patrick Mézard on Wednesday, October 03, 2007 at 11:57:35 +0200
>> Does the following work for you ?
> 
> In principle yes, but it doesn't apply cleanly, as you seem to
> have changed test-convert-cvs.out beforehand to comply with cvs
> 1.11.20.

Ah sorry, I am trying to fix crew-stable, not -stable.

> 
>> I think it is a bit simpler, and it works for me with 1.11.20
>> and 1.11.22.
> 
> Ok. I wasn't sure how much to remove from the output.

Well, we are not really testing CVS here. We just want to have some 
feedback about the files being committed, as well as debugging context 
the test were to failed. I guess listing the files is enough.

> BTW, what is faster:
> 
> grep -F '<--' | sed -e 's:.*src/\(.*\),v.*:checking in src/\1,v:g'
> or
> sed -e '/<--/!d' -e 's:.*src/\(.*\),v.*:checking in src/\1,v:g'
> ?

I don't really know. I am only concerned about portability here, since 
these commands are not issued in a loop. I think the first one is better 
in this regard. Just one question, is there any reason for the "-F" 
optimizations aside ?

BTW, thanks for the report and the patch !

--
Patrick Mézard


More information about the Mercurial-devel mailing list