httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rj...@apache.org
Subject svn commit: r1239026 - in /httpd/httpd/trunk/build: install.sh instdso.sh
Date Wed, 01 Feb 2012 09:45:12 GMT
Author: rjung
Date: Wed Feb  1 09:45:12 2012
New Revision: 1239026

URL: http://svn.apache.org/viewvc?rev=1239026&view=rev
Log:
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.

Modified:
    httpd/httpd/trunk/build/install.sh
    httpd/httpd/trunk/build/instdso.sh

Modified: httpd/httpd/trunk/build/install.sh
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/build/install.sh?rev=1239026&r1=1239025&r2=1239026&view=diff
==============================================================================
--- httpd/httpd/trunk/build/install.sh (original)
+++ httpd/httpd/trunk/build/install.sh Wed Feb  1 09:45:12 2012
@@ -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
 

Modified: httpd/httpd/trunk/build/instdso.sh
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/build/instdso.sh?rev=1239026&r1=1239025&r2=1239026&view=diff
==============================================================================
--- httpd/httpd/trunk/build/instdso.sh (original)
+++ httpd/httpd/trunk/build/instdso.sh Wed Feb  1 09:45:12 2012
@@ -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



Mime
View raw message