common: fix test for ltp/fsstress
authorWang Shilong <wangshilong1991@gmail.com>
Tue, 14 Oct 2014 11:59:38 +0000 (22:59 +1100)
committerDave Chinner <david@fromorbit.com>
Tue, 14 Oct 2014 11:59:38 +0000 (22:59 +1100)
Testing if FSSTRESS_PROG is a null string dosen't make sense because
it has just been set.

Here fix it by testing if it is an executable file.

Signed-off-by: Wang Shilong <wangshilong1991@gmail.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
common/config

index d68d4d0c5d034deb63b430cc11625d970dc1881c..64506a23ac6b53ddfab7bf76ee6ffe7e555d3493 100644 (file)
@@ -135,7 +135,7 @@ export UMOUNT_PROG="`set_prog_path umount`"
 [ "$UMOUNT_PROG" = "" ] && _fatal "umount not found"
 
 export FSSTRESS_PROG="./ltp/fsstress"
-[ "$FSSTRESS_PROG" = "" ] && _fatal "fsstress not found"
+[ ! -x $FSSTRESS_PROG ] && _fatal "fsstress not found or executable"
 
 export PERL_PROG="`set_prog_path perl`"
 [ "$PERL_PROG" = "" ] && _fatal "perl not found"