init.sh: accommodate even those who specify bogus srcdir manually

* tests/init.sh: Normally, srcdir is guaranteed by automake and
configure-time tests to be sanitized, so that there is no need to
use "$srcdir" in Makefile rules and shell scripts.  Using $srcdir
(with no double quotes) suffices.  However, since tests may be
invoked manually, and since you may explicitly set srcdir to the
name of a directory containing spaces, do quote its uses here.
* tests/test-pread.sh: Likewise.
Suggested by Bruno Haible.

diff --git a/ChangeLog b/ChangeLog
index bcb4340..dacf782 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
 2009-11-26  Jim Meyering  <meyering@redhat.com>
 
+       init.sh: accommodate even those who specify bogus srcdir manually
+       * tests/init.sh: Normally, srcdir is guaranteed by automake and
+       configure-time tests to be sanitized, so that there is no need to
+       use "$srcdir" in Makefile rules and shell scripts.  Using $srcdir
+       (with no double quotes) suffices.  However, since tests may be
+       invoked manually, and since you may explicitly set srcdir to the
+       name of a directory containing spaces, do quote its uses here.
+       * tests/test-pread.sh: Likewise.
+       Suggested by Bruno Haible.
+
        test-pread.sh: avoid diagnostics for those who ignore SIGPIPE
        * tests/test-pread.sh: Write no data into the pipe, because
        test-pread actually reads none.  This avoids a diagnostic,

diff --git a/tests/init.sh b/tests/init.sh
index 7d4879f..db2fa76 100644 (file)
--- a/tests/init.sh
+++ b/tests/init.sh
@@ -210,7 +210,7 @@ mktempd_()
 
 # If you want to override the testdir_prefix_ function,
 # or to add more utility functions, use this file.
-test -f $srcdir/init.cfg \
-  && . $srcdir/init.cfg
+test -f "$srcdir/init.cfg" \
+  && . "$srcdir/init.cfg"
 
 setup_ "$@"

diff --git a/tests/test-pread.sh b/tests/test-pread.sh
index d7824ad..59f8536 100755 (executable)
--- a/tests/test-pread.sh
+++ b/tests/test-pread.sh
@@ -1,6 +1,6 @@
 #!/bin/sh
 : ${srcdir=.}
-. $srcdir/init.sh --set-path=.
+. "$srcdir/init.sh" --set-path=.
 
 fail=0
 : | test-pread || fail=1