{"thread":{"id":"14456","subject":"[PATCH] t9600: allow testing with cvsps 2.2, including beta versions","startedAt":"2008-07-15T00:20:19Z","lastAt":"2008-07-16T07:11:47Z","messageCount":4,"participants":["Pavel Roskin","Junio C Hamano","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"83325","messageId":"20080715002019.5337.81984.stgit@dv.roinet.com","threadId":"14456","inReplyTo":null,"subject":"[PATCH] t9600: allow testing with cvsps 2.2, including beta versions","fromName":"Pavel Roskin","fromEmail":"proski@gnu.org","sentAt":"2008-07-15T00:20:19Z","receivedAt":"2008-07-15T00:20:19Z","isPatch":true,"sender":{"key":"proski@gnu.org","avatar":null},"body":"Don't assume that unsupported versions are too old, they may be too new.\n\nSigned-off-by: Pavel Roskin <proski@gnu.org>\n---\n\n t/t9600-cvsimport.sh |    4 +++-\n 1 files changed, 3 insertions(+), 1 deletions(-)\n\ndiff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh\nindex 655f882..f92b47a 100755\n--- a/t/t9600-cvsimport.sh\n+++ b/t/t9600-cvsimport.sh\n@@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps version //p'`\n case \"$cvsps_version\" in\n 2.1)\n \t;;\n+2.2*)\n+\t;;\n '')\n \tsay 'skipping cvsimport tests, cvsps not found'\n \ttest_done\n \texit\n \t;;\n *)\n-\tsay 'skipping cvsimport tests, cvsps too old'\n+\tsay 'skipping cvsimport tests, unsupported cvsps version'\n \ttest_done\n \texit\n \t;;\n"},{"id":"83327","messageId":"7v1w1wdm37.fsf@gitster.siamese.dyndns.org","threadId":"14456","inReplyTo":"20080715002019.5337.81984.stgit@dv.roinet.com","subject":"Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-07-15T00:31:40Z","receivedAt":"2008-07-15T00:31:40Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Pavel Roskin <proski@gnu.org> writes:\n\n> Don't assume that unsupported versions are too old, they may be too new.\n\nExcuse me, but we never assumed such.  \n\nNeither too old nor too new ones are proven to work with us yet, so we\ncannot be call them supported.\n\nLet's apply your patch and see how well version 2.2 fares.\n\nThanks.\n\n> diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh\n> index 655f882..f92b47a 100755\n> --- a/t/t9600-cvsimport.sh\n> +++ b/t/t9600-cvsimport.sh\n> @@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps version //p'`\n>  case \"$cvsps_version\" in\n>  2.1)\n>  \t;;\n> +2.2*)\n> +\t;;\n\nHmm, I would have written:\n\n\tcase \"$...\" in\n        2.1 | 2.2 )\n\nIs the reason why you have the asterisk there because the beta one claims\n\"2.2beta\" or something (I am not suggesting to tighten the match, just\nasking for information)?\n"},{"id":"83328","messageId":"20080714205114.cnwynr1q804wow4k-cebfxv@webmail.spamcop.net","threadId":"14456","inReplyTo":"7v1w1wdm37.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions","fromName":"Pavel Roskin","fromEmail":"proski@gnu.org","sentAt":"2008-07-15T00:51:14Z","receivedAt":"2008-07-15T00:51:14Z","isPatch":true,"sender":{"key":"proski@gnu.org","avatar":null},"body":"Quoting Junio C Hamano <gitster@pobox.com>:\n\n> Pavel Roskin <proski@gnu.org> writes:\n>\n>> Don't assume that unsupported versions are too old, they may be too new.\n>\n> Excuse me, but we never assumed such.\n>\n> Neither too old nor too new ones are proven to work with us yet, so we\n> cannot be call them supported.\n\nThe message was \"skipping cvsimport tests, cvsps too old\" so the  \nassumption was there.\n\n> Let's apply your patch and see how well version 2.2 fares.\n>\n> Thanks.\n>\n>> diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh\n>> index 655f882..f92b47a 100755\n>> --- a/t/t9600-cvsimport.sh\n>> +++ b/t/t9600-cvsimport.sh\n>> @@ -20,13 +20,15 @@ cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps  \n>>  version //p'`\n>>  case \"$cvsps_version\" in\n>>  2.1)\n>>  \t;;\n>> +2.2*)\n>> +\t;;\n>\n> Hmm, I would have written:\n>\n> \tcase \"$...\" in\n>         2.1 | 2.2 )\n\nFine with me, but please allow for extra specifiers.\n\n> Is the reason why you have the asterisk there because the beta one claims\n> \"2.2beta\" or something (I am not suggesting to tighten the match, just\n> asking for information)?\n\nIt's cvsps 2.2b1 from Fedora Development (cvsps-2.2-0.1.b1.fc10.x86_64)\n\n$ cvsps -h 2>&1 | grep version\ncvsps version 2.2b1\n\n-- \nRegards,\nPavel Roskin\n"},{"id":"83483","messageId":"20080716071147.GC4030@segfault.peff.net","threadId":"14456","inReplyTo":"20080714205114.cnwynr1q804wow4k-cebfxv@webmail.spamcop.net","subject":"Re: [PATCH] t9600: allow testing with cvsps 2.2, including beta versions","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2008-07-16T07:11:47Z","receivedAt":"2008-07-16T07:11:47Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Jul 14, 2008 at 08:51:14PM -0400, Pavel Roskin wrote:\n\n> > Excuse me, but we never assumed such.\n> >\n> > Neither too old nor too new ones are proven to work with us yet, so we\n> > cannot be call them supported.\n> \n> The message was \"skipping cvsimport tests, cvsps too old\" so the  \n> assumption was there.\n\nHeh, you can blame me for making that assumption. But at the time I\nwrote it, I think cvsps development had stagnated, so there weren't new\nversions. :)\n\n-Peff\n"}]}