diff options
author | Andreas Gruenbacher <agruenba@redhat.com> | 2018-02-03 19:57:35 +0100 |
---|---|---|
committer | Eryu Guan <eguan@redhat.com> | 2018-02-05 18:00:44 +0800 |
commit | ebfd5ba86ada74bb5d16e502ec8382184d639de2 (patch) | |
tree | 77348d5c6f95931917527a1517aa4551cd70da7e | |
parent | 27f7ed634816fb4748bacbf5f00adb21ef10b0cb (diff) | |
download | xfstests-dev-ebfd5ba86ada74bb5d16e502ec8382184d639de2.tar.gz |
check: Fix exit status
Exit with status 1 if there were any failures: using the number of
failed tests as the exit status makes no sense and can easily
overwrap.
In addition, clean up a few minor things.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Reviewed-by: Eryu Guan <eguan@redhat.com>
Signed-off-by: Eryu Guan <eguan@redhat.com>
-rwxr-xr-x | check | 6 |
1 files changed, 3 insertions, 3 deletions
@@ -194,6 +194,7 @@ trim_test_list() done grep -v -f $tmp.grep <$tmp.list >$tmp.tmp mv $tmp.tmp $tmp.list + rm -f $tmp.grep } @@ -250,7 +251,7 @@ _prepare_test_list() # sort the list of tests into numeric order list=`sort -n $tmp.list | uniq` - rm -f $tmp.list $tmp.tmp $tmp.grep + rm -f $tmp.list if $randomize then @@ -803,7 +804,6 @@ for section in $HOST_OPTIONS_SECTIONS; do then bad="$bad $seqnum" n_bad=`expr $n_bad + 1` - quick=false tc_status="fail" fi if $do_report; then @@ -822,5 +822,5 @@ for section in $HOST_OPTIONS_SECTIONS; do done interrupt=false -status=`expr $sum_bad` +status=`expr $sum_bad != 0` exit |