This is an automated email from the git hooks/post-receive script.

plessy pushed a commit to branch debian/unstable
in repository samtools.

commit f11d48b7ee62c33b39e2d12ec5467fe257bbfa31
Merge: eabd608 6f27dcd
Author: John Marshall <[email protected]>
Date:   Wed Dec 4 09:10:40 2013 +0000

    Fail when tests fail
    
    Exit statuses are really 8-bit unsigned integers, so exit with 0/1
    rather than the number of failures in case we have lots of tests, and
    return a less unusual status from the usage display: 1 rather than 255.
    
    Merge branch bugfix/test-pl-no-exit-code-on-fail of 
github.com/wtsi-hgi/samtools

 test/test.pl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --cc test/test.pl
index e0159d2,f3cd5b7..0654640
--- a/test/test.pl
+++ b/test/test.pl
@@@ -21,7 -20,7 +21,7 @@@ printf "    passed  .. %d\n", $$opts{no
  printf "    failed  .. %d\n", $$opts{nfailed};
  print "\n";
  
- exit;
 -exit $$opts{nfailed};
++exit ($$opts{nfailed} > 0);
  
  #--------------------
  
@@@ -37,7 -36,7 +37,7 @@@ sub erro
          "   -t, --temp-dir <path>           When given, temporary files will 
not be removed.\n",
          "   -h, -?, --help                  This help message.\n",
          "\n";
--    exit -1;
++    exit 1;
  }
  sub parse_params
  {

-- 
Alioth's /usr/local/bin/git-commit-notice on 
/srv/git.debian.org/git/debian-med/samtools.git

_______________________________________________
debian-med-commit mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-med-commit

Reply via email to