On Wed, Feb 5, 2014 at 12:17 PM, Michael Haggerty <mhag...@alum.mit.edu> wrote: > On 02/05/2014 11:44 AM, László Böszörményi (GCS) wrote: >> Just tried again. Version 2.4.0 with the patch applied still fails with: >> -- cut -- >> PASS: run-tests.py 3: generate a manpage for cvs2git >> SKIP: run-tests.py 4: generate a manpage for cvs2hg >> unexpected log output (missing changed paths) >> Line: ' >> ' >> EXCEPTION: SystemExit(1), skipping cleanup >> FAIL: run-tests.py 5: detection of the executable flag >> -- cut -- > > This test works for me. > > My guess is that either the package itself or the test's temporary files > are on a filesystem that does not allow the executable bit to be set. Not the case. I've one partition for everything (this is a test environment). $ mount | grep ' / ' /dev/sdb1 on / type ext4 (rw,errors=remount-ro) $ mount | grep noexec [ shows the usual drill, /proc , /sys , /run and /dev/pts ] ~/test/cvs2svn-2.4.0 $ rm -f ./test.sh && echo 'echo "works!"' >test.sh && chmod a+x ./test.sh && ./test.sh works!
In short, the fs has the executable bit allowed (otherwise my whole system would just break), and testing +x with a shell script in the cvs2svn build directory succedded. Will try to get a more verbose output of the python test, why it fails. > So if indeed the Debian test infrastructure does not allow the > executable bit to be set, the only alternative would be to skip this > test on your setup. If the limitation is on the input then something > like this should do the trick: [...] > If the limitation is on the output directory, then we would probably > have to try setting the executable bit on some file to see if it is allowed. None the case, see the tests above. Especially that the mentioned file is: ~/test/cvs2svn-2.4.0 # ls -l ./test-data/main-cvsrepos/single-files/attr-exec,v -rwxr-xr-x 1 1000 1000 424 Oct 8 2007 ./test-data/main-cvsrepos/single-files/attr-exec,v Thus it _has_ the a+x set. First I thought the missing go+w bits may cause problems. Then # chmod 0777 ./test-data/main-cvsrepos/single-files/attr-exec,v # rerun the build and test process, still fails the exact same way # ls -l ./test-data/main-cvsrepos/single-files/attr-exec,v -rwxrwxrwx 1 1000 1000 424 Oct 8 2007 ./test-data/main-cvsrepos/single-files/attr-exec,v Now it has all rights a normal user may have, but still can't get that the +x is indeed set on that file. Laszlo/GCS -- To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
