Merge remote branch 'branchable/master'

This commit is contained in:
Joey Hess 2011-02-13 14:22:59 -04:00
commit 2b1059032b
4 changed files with 88 additions and 0 deletions

View file

@ -0,0 +1,43 @@
[[!comment format=mdwn
username="https://www.google.com/accounts/o8/id?id=AItOawkSq2FDpK2n66QRUxtqqdbyDuwgbQmUWus"
nickname="Jimmy"
subject="comment 17"
date="2011-02-13T10:46:54Z"
content="""
Yeap, that did the trick. I just tested a few separate OSX 10.6.6 systems and the tests are better behaved now, only 3 failures now.
So the tests behave better (at least we don't get resource fork errors any more)
* after the commit c319a3 without modifying the system limits (of 266 procs per user)
* without the commit c319a3 and when I increase the system process limits to as much as OSX allows
On all the systems I tested on, I'm down to 3 failures now.
<pre>
### Failure in: 1:blackbox:3:git-annex unannex:1:with content
foo is not a symlink
### Failure in: 1:blackbox:4:git-annex drop:0:no remotes
drop wrongly succeeded with no known copy of file
Cases: 30 Tried: 20 Errors: 0 Failures: 2add foo ok
ok
Cases: 30 Tried: 24 Errors: 0 Failures: 2 Only 1 of 2 trustworthy copies of foo exist.
Back it up with git-annex copy.
Only 1 of 2 trustworthy copies of sha1foo exist.
Back it up with git-annex copy.
Bad file size; moved to /Users/jtang/develop/git-annex/.t/tmprepo/.git/annex/bad/WORM:1297594011:20:foo
Bad file content; moved to /Users/jtang/develop/git-annex/.t/tmprepo/.git/annex/bad/SHA1:ee80d2cec57a3810db83b80e1b320df3a3721ffa
### Failure in: 1:blackbox:12:git-annex fsck:1
fsck failed to fail with content only available in untrusted (current) repository
Cases: 30 Tried: 26 Errors: 0 Failures: 3 Only 1 of 2 trustworthy copies of foo exist.
Back it up with git-annex copy.
The following untrusted locations may also have copies:
90d63906-375e-11e0-8867-abb8a6368269 -- test repo
Only 1 of 2 trustworthy copies of sha1foo exist.
Back it up with git-annex copy.
The following untrusted locations may also have copies:
90d63906-375e-11e0-8867-abb8a6368269 -- test repo
Cases: 30 Tried: 30 Errors: 0 Failures: 3
</pre>
It's the same set of failures across all the OSX systems that I have tested on. Now I just need to figure out why there are still these three failures.
"""]]

View file

@ -0,0 +1,24 @@
[[!comment format=mdwn
username="https://www.google.com/accounts/o8/id?id=AItOawkSq2FDpK2n66QRUxtqqdbyDuwgbQmUWus"
nickname="Jimmy"
subject="maybe killed another osx bug in the test."
date="2011-02-13T15:12:10Z"
content="""
I think I have figured out why
### Failure in: 1:blackbox:3:git-annex unannex:1:with content
foo is not a symlink
It goes back to the this piece of code (in test.hs)
copyrepo :: FilePath -> FilePath -> IO FilePath
copyrepo old new = do
cleanup new
ensuretmpdir
Utility.boolSystem \"cp\" [\"-pr\", old, new] @? \"cp -pr failed\"
It seems that on OSX it does not preserve the symbolic link information, basically cp is not gnu cp on OSX, doing a \"cp -a SOURCE DEST\" seem's to the right thing on OSX. I tried it out on my archlinux workstation by replacing *-pr* with just *-a* and all the tests passed on archlinux.
I'm not sure what the implications would be with changing the test with changing the cp command.
"""]]

View file

@ -0,0 +1,11 @@
[[!comment format=mdwn
username="https://www.google.com/accounts/o8/id?id=AItOawkSq2FDpK2n66QRUxtqqdbyDuwgbQmUWus"
nickname="Jimmy"
subject="comment 19"
date="2011-02-13T15:55:47Z"
content="""
On second thought and after some messing (trying most of the options and combinations of options on OSX for).... I tried replacing cp with gnu cp from coreutils on my OSX install, and all the tests passed. *sigh* cp -a is preserving some permissions and attributes but not all, its not behaving in the same way as the gnu cp does... the closet thing that I have found on OSX that behaves in the same way as gnu \"cp -pr\" is to use \"ditto\".
Just doing a \"ditto SOURCE DEST\" in the tests passes everything. I'm not sure if its a good idea to use this even though it works. Though this is just the tests, does it affect CopyFile.hs where \"cp\" is called?
"""]]

View file

@ -0,0 +1,10 @@
[[!comment format=mdwn
username="http://joey.kitenet.net/"
nickname="joey"
subject="comment 20"
date="2011-02-13T17:54:09Z"
content="""
Outside the test suite, git-annex's actual use of cp puts fairly low demands on it. It tries to use cp -a or cp -p if available just to preserve whatever attributes it can preserve, but the worst case if that you have a symlink pointing to a file that doesn't have the original timestamp or whatever. And there's little expectation git preserves that stuff anyway.
I will probably try to make the test suite entirely use git clone rather than cp.
"""]]