Addition to r1237447: "for arg ; do" is not

sh standard. It seems its a shortcut for
"for arg in "$@" ; do".

Detected on Solaris with very restricted /bin/sh.

Seems to work fine for arguments with embedded
whitespace.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1239026 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/build/install.sh b/build/install.sh
index 21bc16c2d2093b200d2e648d9509cc879e88b11a..24488475d645160f2362affc5bb71b117312cca1 100755 (executable)
--- a/build/install.sh
+++ b/build/install.sh
@@ -83,7 +83,7 @@ if test $# -eq 1 ; then
     echo "install.sh: no destination specified"
     exit 1
 fi
-for arg ; do
+for arg in "$@" ; do
     dstarg="$arg"
 done
 

diff --git a/build/instdso.sh b/build/instdso.sh
index 405b304c9a770b8027d50c8a16916d08e459c08e..89aa32d010840342fd6f1c0c8d21f4537a96b7c5 100755 (executable)
--- a/build/instdso.sh
+++ b/build/instdso.sh
@@ -33,7 +33,7 @@ fi
 SH_LIBTOOL=`echo $1 | sed -e 's/^SH_LIBTOOL=//'`
 shift
 # get last arg
-for arg ; do
+for arg in "$@" ; do
     DSOARCHIVES="$DSOARCHIVES $TARGETDIR"
     TARGETDIR=$arg
 done