diff mbox

xfstests/common: fix to test if ltp/fsstress valid

Message ID 1412613730-53245-1-git-send-email-wangshilong1991@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Wang Shilong Oct. 6, 2014, 4:42 p.m. UTC 
test 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>
---
 common/config | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox

Patch

diff --git a/common/config b/common/config
index d68d4d0..64506a2 100644
--- a/common/config
+++ b/common/config
@@ -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"