Unquoted $ORIGDIR in autogen.sh

https://bugs.webkit.org/show_bug.cgi?id=19512

Reviewed by Carlos Garcia Campos.

Quote all directory names. This doesn't mean that all possible
directory names are safe for building webkit, but the configure
script already runs a sanity check.

* autogen.sh:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155515 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/ChangeLog b/ChangeLog
index 2ca7ff8..3118852 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2013-09-11  Alberto Garcia  <berto@igalia.com>
+
+        Unquoted $ORIGDIR in autogen.sh
+        https://bugs.webkit.org/show_bug.cgi?id=19512
+
+        Reviewed by Carlos Garcia Campos.
+
+        Quote all directory names. This doesn't mean that all possible
+        directory names are safe for building webkit, but the configure
+        script already runs a sanity check.
+
+        * autogen.sh:
+
 2013-09-11  Zan Dobersek  <zdobersek@igalia.com>
 
         [GTK] Stop disabling deprecated symbols in debug builds

diff --git a/autogen.sh b/autogen.sh
index 12c6a4c..c6688d4 100755 (executable)
--- a/autogen.sh
+++ b/autogen.sh
@@ -2,11 +2,11 @@
 
 # Allow invocation from a separate build directory; in that case, we change
 # to the source directory to run the auto*, then change back before running configure
-srcdir=`dirname $0`
+srcdir=`dirname "$0"`
 test -z "$srcdir" && srcdir=.
 
 ORIGDIR=`pwd`
-cd $srcdir
+cd "$srcdir" || exit 1
 
 rm -f $top_srcdir/autom4te.cache
 
@@ -18,9 +18,9 @@ if test -z `which autoreconf`; then
 fi
 autoreconf --verbose --install -I Source/autotools $ACLOCAL_FLAGS|| exit $?
 
-cd $ORIGDIR || exit 1
+cd "$ORIGDIR" || exit 1
 
 if test -z "$NOCONFIGURE"; then
-    $srcdir/configure $AUTOGEN_CONFIGURE_ARGS "$@" || exit $?
+    "$srcdir"/configure $AUTOGEN_CONFIGURE_ARGS "$@" || exit $?
 fi