* [PATCH] t9600: allow testing with cvsps 2.2, including beta versions
@ 2008-07-15 0:20 Pavel Roskin
2008-07-15 0:31 ` Junio C Hamano
0 siblings, 1 reply; 4+ messages in thread
From: Pavel Roskin @ 2008-07-15 0:20 UTC (permalink / raw)
To: git
Don't assume that unsupported versions are too old, they may be too new.
Signed-off-by: Pavel Roskin <proski@gnu•org>
---
t/t9600-cvsimport.sh | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh
index 655f882..f92b47a 100755
--- a/t/t9600-cvsimport.sh
+++ b/t/t9600-cvsimport.sh
@@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps version //p'`
case "$cvsps_version" in
2.1)
;;
+2.2*)
+ ;;
'')
say 'skipping cvsimport tests, cvsps not found'
test_done
exit
;;
*)
- say 'skipping cvsimport tests, cvsps too old'
+ say 'skipping cvsimport tests, unsupported cvsps version'
test_done
exit
;;
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions
2008-07-15 0:20 [PATCH] t9600: allow testing with cvsps 2.2, including beta versions Pavel Roskin
@ 2008-07-15 0:31 ` Junio C Hamano
2008-07-15 0:51 ` Pavel Roskin
0 siblings, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2008-07-15 0:31 UTC (permalink / raw)
To: Pavel Roskin; +Cc: git
Pavel Roskin <proski@gnu•org> writes:
> Don't assume that unsupported versions are too old, they may be too new.
Excuse me, but we never assumed such.
Neither too old nor too new ones are proven to work with us yet, so we
cannot be call them supported.
Let's apply your patch and see how well version 2.2 fares.
Thanks.
> diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh
> index 655f882..f92b47a 100755
> --- a/t/t9600-cvsimport.sh
> +++ b/t/t9600-cvsimport.sh
> @@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps version //p'`
> case "$cvsps_version" in
> 2.1)
> ;;
> +2.2*)
> + ;;
Hmm, I would have written:
case "$..." in
2.1 | 2.2 )
Is the reason why you have the asterisk there because the beta one claims
"2.2beta" or something (I am not suggesting to tighten the match, just
asking for information)?
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions
2008-07-15 0:31 ` Junio C Hamano
@ 2008-07-15 0:51 ` Pavel Roskin
2008-07-16 7:11 ` Jeff King
0 siblings, 1 reply; 4+ messages in thread
From: Pavel Roskin @ 2008-07-15 0:51 UTC (permalink / raw)
To: Junio C Hamano; +Cc: git
Quoting Junio C Hamano <gitster@pobox•com>:
> Pavel Roskin <proski@gnu•org> writes:
>
>> Don't assume that unsupported versions are too old, they may be too new.
>
> Excuse me, but we never assumed such.
>
> Neither too old nor too new ones are proven to work with us yet, so we
> cannot be call them supported.
The message was "skipping cvsimport tests, cvsps too old" so the
assumption was there.
> Let's apply your patch and see how well version 2.2 fares.
>
> Thanks.
>
>> diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh
>> index 655f882..f92b47a 100755
>> --- a/t/t9600-cvsimport.sh
>> +++ b/t/t9600-cvsimport.sh
>> @@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps
>> version //p'`
>> case "$cvsps_version" in
>> 2.1)
>> ;;
>> +2.2*)
>> + ;;
>
> Hmm, I would have written:
>
> case "$..." in
> 2.1 | 2.2 )
Fine with me, but please allow for extra specifiers.
> Is the reason why you have the asterisk there because the beta one claims
> "2.2beta" or something (I am not suggesting to tighten the match, just
> asking for information)?
It's cvsps 2.2b1 from Fedora Development (cvsps-2.2-0.1.b1.fc10.x86_64)
$ cvsps -h 2>&1 | grep version
cvsps version 2.2b1
--
Regards,
Pavel Roskin
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions
2008-07-15 0:51 ` Pavel Roskin
@ 2008-07-16 7:11 ` Jeff King
0 siblings, 0 replies; 4+ messages in thread
From: Jeff King @ 2008-07-16 7:11 UTC (permalink / raw)
To: Pavel Roskin; +Cc: Junio C Hamano, git
On Mon, Jul 14, 2008 at 08:51:14PM -0400, Pavel Roskin wrote:
> > Excuse me, but we never assumed such.
> >
> > Neither too old nor too new ones are proven to work with us yet, so we
> > cannot be call them supported.
>
> The message was "skipping cvsimport tests, cvsps too old" so the
> assumption was there.
Heh, you can blame me for making that assumption. But at the time I
wrote it, I think cvsps development had stagnated, so there weren't new
versions. :)
-Peff
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-07-16 7:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-07-15 0:20 [PATCH] t9600: allow testing with cvsps 2.2, including beta versions Pavel Roskin
2008-07-15 0:31 ` Junio C Hamano
2008-07-15 0:51 ` Pavel Roskin
2008-07-16 7:11 ` Jeff King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox