[med-svn] samtools 19/26: Fail when tests fail

Charles Plessy plessy at moszumanska.debian.org
Tue Dec 10 10:06:26 UTC 2013


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 <jm18 at sanger.ac.uk>
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



More information about the debian-med-commit mailing list