Re: make test failure with latest master
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Dec 3, 2007, 23:21 UTC
- Message-ID
- <7vir3fe54c.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <47548CEC.6010701@gmail.com>
A Large Angry SCM <gitzilla@gmail.com> writes:
Show 32 quoted lines
> Jeff King wrote: >> On Sun, Dec 02, 2007 at 11:41:10PM +0000, Johannes Schindelin wrote: >> >>>> Attached is the output of >>>> >>>> ./t9600-cvsimport.sh --verbose >/tmp/9600_out.txt 2>&1 >>>> >>> Unfortunately you attached the interesting part, so I cannot quote >>> it here. Seems that your cvsps does not understand the "-A" >>> option. So it looks like it is too old. > > [Ignoring gitte's declaration that the core of git is dependent on CVS] > >> Yes, the changelog indicates that -A was added in 2.1, and he is running >> 2.0rc1. >> >> We can do something like this, though it feels a bit hack-ish and will >> need updated for new cvsps versions (a less lazy or more clever shell >> coder than me could do a nice >= version comparator). > > The patch "fixes" the issue for me. > >> -- >8 -- >> t9600: require cvsps 2.1 to perform tests >> >> git-cvsimport won't run at all with less than cvsps 2.1, because it >> lacks the -A flag. But there's no point in preventing people who have an >> old cvsps from running the full testsuite. >> >> Signed-off-by: Jeff King <peff@peff.net> > > Acked-by: A Large Angry SCM <gitzilla@gmail.com>
It would be Tested-by: in a stricter sense of the word but thanks for testing.
Jeff, would you want to further amend the patch to use "say" as Hannes suggested to make it look nicer?