Mike Ralphson wrote:
> Hi Nick, I'm seeing intermittent failures since your new test was
> added to 'next' on AIX 5.3
>
> cvs commit: Up-to-date check failed for ` space'
> cvs [commit aborted]: correct above errors first!
> * FAIL 15: re-commit a removed filename which remains in CVS attic
> * failed 1 among 15 test(s)
>
> Is there a possibility this test has a race condition?
Hm, I have thought not, but what sort of a race condition did you have in mind?
> Let me know if there's anything I can do to help debug it.
>
> It could be a bug in the ancient CVS I have here (1.11.1p1) though.
I wouldn't be surprised.
What my fix does is to spot files marked "Up-to-date" but which have "no file "
prepended to the filename in cvs status's output, which means the file used to
exist but was deleted from the repository, not that it is "Up-to-date" in the
sense you'd expect.
If old versions of CVS report these files differently they may not be spotted.
I don't know why the " space" file seems to be causing a problem - it shouldn't
interfere with the test I added (and indeed doesn't for me).
Perhaps you could apply the following patch to t/t9200-git-cvsexportcommit.sh,
run it, and send the contents of t/debug.out? What I get is appended after the
patch.
You might also try commenting out the following part of my test, so that it
should trivially work, and see if there's still an error:
# rm attic_gremlin &&
# cvs -Q rm attic_gremlin &&
# cvs -Q ci -m "removed attic_gremlin"
Cheers,
N
diff --git a/t/t9200-git-cvsexportcommit.sh b/t/t9200-git-cvsexportcommit.sh
index ef1f8d2..4f19a47 100755
--- a/t/t9200-git-cvsexportcommit.sh
+++ b/t/t9200-git-cvsexportcommit.sh
@@ -320,12 +320,19 @@ test_expect_success 'use the same checkout for Git and CVS' '
test_expect_success 're-commit a removed filename which remains in CVS attic' '
(cd "$CVSWORK" &&
+ dlog=../../debug.out &&
+ echo "# before adding file" >$dlog &&
+ cvs status attic_gremlin " space" >>$dlog 2>&1 &&
+
echo >attic_gremlin &&
cvs -Q add attic_gremlin &&
cvs -Q ci -m "added attic_gremlin" &&
rm attic_gremlin &&
cvs -Q rm attic_gremlin &&
- cvs -Q ci -m "removed attic_gremlin") &&
+ cvs -Q ci -m "removed attic_gremlin" &&
+
+ echo "# after adding file" >>$dlog &&
+ cvs status attic_gremlin " space" >>$dlog 2>&1) &&
echo > attic_gremlin &&
git add attic_gremlin &&
----
# before adding file
cvs status: nothing known about `attic_gremlin'
===================================================================
File: no file attic_gremlin Status: Unknown
Working revision: No entry for attic_gremlin
Repository revision: No revision control file
===================================================================
File: space Status: Up-to-date
Working revision: 1.1 2009-06-11 13:56:15 +0000
Repository revision: 1.1 /home/nick/gitworking/git/t/trash
directory.t9200-git-cvsexportcommit/cvsroot/ space,v
Commit Identifier: oDOEzyj3FzAazrRt
Sticky Tag: (none)
Sticky Date: (none)
Sticky Options: (none)
# after adding file
===================================================================
File: no file attic_gremlin Status: Up-to-date
Working revision: No entry for attic_gremlin
Repository revision: 1.2 /home/nick/gitworking/git/t/trash
directory.t9200-git-cvsexportcommit/cvsroot/Attic/attic_gremlin,v
Commit Identifier: X1mPOnzvk5rczrRt
===================================================================
File: space Status: Up-to-date
Working revision: 1.1 2009-06-11 13:56:15 +0000
Repository revision: 1.1 /home/nick/gitworking/git/t/trash
directory.t9200-git-cvsexportcommit/cvsroot/ space,v
Commit Identifier: oDOEzyj3FzAazrRt
Sticky Tag: (none)
Sticky Date: (none)
Sticky Options: (none)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html