* [PATCH v2 1/3] t/lib-cvs.sh: allow cvsps version 3.x.
2013-01-12 6:21 [PATCH v2 0/3] fixup remaining cvsimport tests Chris Rorvick
@ 2013-01-12 6:21 ` Chris Rorvick
2013-01-12 6:21 ` [PATCH v2 2/3] t9600: fixup for new cvsimport Chris Rorvick
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Chris Rorvick @ 2013-01-12 6:21 UTC (permalink / raw)
To: git; +Cc: Eric S. Raymond, Junio C Hamano, Chris Rorvick
Signed-off-by: Chris Rorvick <chris@rorvick.com>
---
t/lib-cvs.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/lib-cvs.sh b/t/lib-cvs.sh
index 44263ad..b55e861 100644
--- a/t/lib-cvs.sh
+++ b/t/lib-cvs.sh
@@ -15,7 +15,7 @@ export CVS
cvsps_version=`cvsps -h 2>&1 | sed -ne 's/cvsps version //p'`
case "$cvsps_version" in
-2.1 | 2.2*)
+2.1 | 2.2* | 3.*)
;;
'')
skip_all='skipping cvsimport tests, cvsps not found'
--
1.8.1.rc3.335.g88a67d6
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH v2 2/3] t9600: fixup for new cvsimport
2013-01-12 6:21 [PATCH v2 0/3] fixup remaining cvsimport tests Chris Rorvick
2013-01-12 6:21 ` [PATCH v2 1/3] t/lib-cvs.sh: allow cvsps version 3.x Chris Rorvick
@ 2013-01-12 6:21 ` Chris Rorvick
2013-01-12 6:21 ` [PATCH v2 3/3] t9604: " Chris Rorvick
2013-01-12 6:36 ` [PATCH v2 0/3] fixup remaining cvsimport tests Junio C Hamano
3 siblings, 0 replies; 6+ messages in thread
From: Chris Rorvick @ 2013-01-12 6:21 UTC (permalink / raw)
To: git; +Cc: Eric S. Raymond, Junio C Hamano, Chris Rorvick
cvsimport no longer supports -a (import all commits including recent ones)
and no longer uses the 'origin' branch by default for imports.
Signed-off-by: Chris Rorvick <chris@rorvick.com>
---
t/t9600-cvsimport.sh | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/t/t9600-cvsimport.sh b/t/t9600-cvsimport.sh
index 4c384ff..14f54d5 100755
--- a/t/t9600-cvsimport.sh
+++ b/t/t9600-cvsimport.sh
@@ -44,7 +44,7 @@ EOF
test_expect_success PERL 'import a trivial module' '
- git cvsimport -a -R -z 0 -C module-git module &&
+ git cvsimport -R -z 0 -C module-git module &&
test_cmp module-cvs/o_fortuna module-git/o_fortuna
'
@@ -90,8 +90,7 @@ test_expect_success PERL 'update git module' '
(cd module-git &&
git config cvsimport.trackRevisions true &&
- git cvsimport -a -z 0 module &&
- git merge origin
+ git cvsimport -z 0 module
) &&
test_cmp module-cvs/o_fortuna module-git/o_fortuna
@@ -119,8 +118,7 @@ test_expect_success PERL 'cvsimport.module config works' '
(cd module-git &&
git config cvsimport.module module &&
git config cvsimport.trackRevisions true &&
- git cvsimport -a -z0 &&
- git merge origin
+ git cvsimport -z0
) &&
test_cmp module-cvs/tick module-git/tick
@@ -140,7 +138,7 @@ test_expect_success PERL 'import from a CVS working tree' '
$CVS co -d import-from-wt module &&
(cd import-from-wt &&
git config cvsimport.trackRevisions false &&
- git cvsimport -a -z0 &&
+ git cvsimport -z0 &&
echo 1 >expect &&
git log -1 --pretty=format:%s%n >actual &&
test_cmp actual expect
--
1.8.1.rc3.335.g88a67d6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 3/3] t9604: fixup for new cvsimport
2013-01-12 6:21 [PATCH v2 0/3] fixup remaining cvsimport tests Chris Rorvick
2013-01-12 6:21 ` [PATCH v2 1/3] t/lib-cvs.sh: allow cvsps version 3.x Chris Rorvick
2013-01-12 6:21 ` [PATCH v2 2/3] t9600: fixup for new cvsimport Chris Rorvick
@ 2013-01-12 6:21 ` Chris Rorvick
2013-01-12 6:36 ` [PATCH v2 0/3] fixup remaining cvsimport tests Junio C Hamano
3 siblings, 0 replies; 6+ messages in thread
From: Chris Rorvick @ 2013-01-12 6:21 UTC (permalink / raw)
To: git; +Cc: Eric S. Raymond, Junio C Hamano, Chris Rorvick
cvsps no longer writes a cache file and therefore no longer can be told
to ignore it with -x.
Signed-off-by: Chris Rorvick <chris@rorvick.com>
---
t/t9604-cvsimport-timestamps.sh | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/t/t9604-cvsimport-timestamps.sh b/t/t9604-cvsimport-timestamps.sh
index 1fd5142..b1629b6 100755
--- a/t/t9604-cvsimport-timestamps.sh
+++ b/t/t9604-cvsimport-timestamps.sh
@@ -7,8 +7,7 @@ setup_cvs_test_repository t9604
test_expect_success 'check timestamps are UTC (TZ=CST6CDT)' '
- TZ=CST6CDT git cvsimport -p"-x" -C module-1 module &&
- git cvsimport -p"-x" -C module-1 module &&
+ TZ=CST6CDT git cvsimport -C module-1 module &&
(
cd module-1 &&
git log --format="%s %ai"
@@ -42,7 +41,7 @@ test_expect_success 'check timestamps with author-specific timezones' '
user3=User Three <user3@domain.org> EST5EDT
user4=User Four <user4@domain.org> MST7MDT
EOF
- git cvsimport -p"-x" -A cvs-authors -C module-2 module &&
+ git cvsimport -A cvs-authors -C module-2 module &&
(
cd module-2 &&
git log --format="%s %ai %an"
--
1.8.1.rc3.335.g88a67d6
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 0/3] fixup remaining cvsimport tests
2013-01-12 6:21 [PATCH v2 0/3] fixup remaining cvsimport tests Chris Rorvick
` (2 preceding siblings ...)
2013-01-12 6:21 ` [PATCH v2 3/3] t9604: " Chris Rorvick
@ 2013-01-12 6:36 ` Junio C Hamano
2013-01-12 7:01 ` Chris Rorvick
3 siblings, 1 reply; 6+ messages in thread
From: Junio C Hamano @ 2013-01-12 6:36 UTC (permalink / raw)
To: Chris Rorvick; +Cc: git, Eric S. Raymond
Chris Rorvick <chris@rorvick.com> writes:
> Reroll w/ sign-off.
>
> Chris Rorvick (3):
> t/lib-cvs.sh: allow cvsps version 3.x.
> t9600: fixup for new cvsimport
> t9604: fixup for new cvsimport
>
> t/lib-cvs.sh | 2 +-
> t/t9600-cvsimport.sh | 10 ++++------
> t/t9604-cvsimport-timestamps.sh | 5 ++---
> 3 files changed, 7 insertions(+), 10 deletions(-)
Thanks.
I too noticed the droppage of "-a" support, which may not be a big
deal (people can drop it from their script, run cvsimport and they
can drop newer commits from the resulting Git history to emulate the
old behaviour without "-a" that attempted to find a quiescent point
if they really want to and suspect that the upstream CVS repository
was not quiescent during the import).
Likewise for "-x". You said "no longer can be told" and that is
technically true, but it is more like "no longer need to be told, as
stale cache cannot get in the way", so it is probably not a big
deal, either, for people to drop it from their script.
About the changed behaviour regarding "origin", I suspect that it is
a change for the better, but we would probably need documentation
updates to cover it (and deleted options and (mis)features) before
this topic graduates.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v2 0/3] fixup remaining cvsimport tests
2013-01-12 6:36 ` [PATCH v2 0/3] fixup remaining cvsimport tests Junio C Hamano
@ 2013-01-12 7:01 ` Chris Rorvick
0 siblings, 0 replies; 6+ messages in thread
From: Chris Rorvick @ 2013-01-12 7:01 UTC (permalink / raw)
To: Junio C Hamano; +Cc: git, Eric S. Raymond
On Sat, Jan 12, 2013 at 12:36 AM, Junio C Hamano <gitster@pobox.com> wrote:
> I too noticed the droppage of "-a" support, which may not be a big
> deal (people can drop it from their script, run cvsimport and they
> can drop newer commits from the resulting Git history to emulate the
> old behaviour without "-a" that attempted to find a quiescent point
> if they really want to and suspect that the upstream CVS repository
> was not quiescent during the import).
Is there any value to -a when fuzz is exposed (-z)? I mean this is a
functional sense. I think there is a lot of value to maintaining the
interfaces of both cvsimport and cvsps where possible.
> Likewise for "-x". You said "no longer can be told" and that is
> technically true, but it is more like "no longer need to be told, as
> stale cache cannot get in the way", so it is probably not a big
> deal, either, for people to drop it from their script.
:-) I originally wrote "need" and then changed it to be clearer on
why it was being removed.
^ permalink raw reply [flat|nested] 6+ messages in thread