Run the scripts with the correct shell in test_scripts.sh.

The scripts are now made executable in the build tree.
This way the scripts can be run like programs in
test_scripts.sh. Previously test_scripts.sh always
used sh but it's not correct if @POSIX_SHELL@ is set
to something else by configure.

Thanks to Jonathan Nieder for the patch.

diff --git a/configure.ac b/configure.ac
index 39fd0ef4cf75b43b9d47161a920359c1e73d3327..9bc1e89505638ab4141d09f00663369961e091e9 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -627,13 +627,13 @@ AC_CONFIG_FILES([
        src/xzdec/Makefile
        src/lzmainfo/Makefile
        src/scripts/Makefile
-       src/scripts/xzdiff
-       src/scripts/xzgrep
-       src/scripts/xzmore
-       src/scripts/xzless
        tests/Makefile
        debug/Makefile
 ])
+AC_CONFIG_FILES([src/scripts/xzdiff], [chmod +x src/scripts/xzdiff])
+AC_CONFIG_FILES([src/scripts/xzgrep], [chmod +x src/scripts/xzgrep])
+AC_CONFIG_FILES([src/scripts/xzmore], [chmod +x src/scripts/xzmore])
+AC_CONFIG_FILES([src/scripts/xzless], [chmod +x src/scripts/xzless])
 
 AC_OUTPUT
 

diff --git a/tests/test_scripts.sh b/tests/test_scripts.sh
index 891fc76a5eec93d7004516afa0c60b90bfbab9c9..293929e61950f4ce2130bc2ab56dd32ff25afa34 100755 (executable)
--- a/tests/test_scripts.sh
+++ b/tests/test_scripts.sh
@@ -13,7 +13,7 @@
 XZ=../src/xz/xz
 XZDIFF=../src/scripts/xzdiff
 test -x "$XZ" || XZ=
-test -r "$XZDIFF" || XZDIFF=
+test -x "$XZDIFF" || XZDIFF=
 if test -z "$XZ" || test -z "$XZDIFF"; then
        (exit 77)
        exit 77
@@ -26,7 +26,7 @@ preimage=$srcdir/files/good-1-check-crc32.xz
 samepostimage=$srcdir/files/good-1-check-crc64.xz
 otherpostimage=$srcdir/files/good-1-lzma2-1.xz
 
-sh "$XZDIFF" "$preimage" "$samepostimage" >/dev/null
+"$XZDIFF" "$preimage" "$samepostimage" >/dev/null
 status=$?
 if test "$status" != 0 ; then
        echo "xzdiff with no changes exited with status $status != 0"
@@ -34,7 +34,7 @@ if test "$status" != 0 ; then
        exit 1
 fi
 
-sh "$XZDIFF" "$preimage" "$otherpostimage" >/dev/null
+"$XZDIFF" "$preimage" "$otherpostimage" >/dev/null
 status=$?
 if test "$status" != 1 ; then
        echo "xzdiff with changes exited with status $status != 1"
@@ -42,7 +42,7 @@ if test "$status" != 1 ; then
        exit 1
 fi
 
-sh "$XZDIFF" "$preimage" "$srcdir/files/missing.xz" >/dev/null 2>&1
+"$XZDIFF" "$preimage" "$srcdir/files/missing.xz" >/dev/null 2>&1
 status=$?
 if test "$status" != 2 ; then
        echo "xzdiff with missing operand exited with status $status != 2"