* config/ltmain.in: Double quote even $# for the sake of sh.test.

(exec_cmd): Be careful about "`"foo"`" quoting, even in comments
to avoid triggering the tests in sh.test.

diff --git a/ChangeLog b/ChangeLog
index 23ff3e865721e8c783f62ca798d86bb2ea02f4bb..5f20e27071e655b4da8be0b6c3201ba4ee1280ad 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2004-08-30  Gary V. Vaughan  <gary@gnu.org>
 
+       * config/ltmain.in: Double quote even $# for the sake of sh.test.
+       (exec_cmd): Be careful about "`\"foo\"`" quoting, even in comments
+       to avoid triggering the tests in sh.test.
+
        * configure.ac:  Because of the way the autoconf C++ testing
        works, we need to call the C test first to work in an environment
        that has no C++ compiler.

diff --git a/config/ltmain.in b/config/ltmain.in
index fb207ca01ff3a795a2db94798c09f6993ce1d181..ba9c09b570fee47108c1854f920816eacecd50cb 100644 (file)
--- a/config/ltmain.in
+++ b/config/ltmain.in
@@ -370,8 +370,8 @@ func_enable_tag ()
 
        # Evaluate the configuration.  Be careful to quote the path
        # and the sed script, to avoid splitting on whitespace, but
-       # also don't use non-portable "`\"foo\"`", so we have to do
-       # it in 2 steps:
+       # also don't use non-portable quotes within backquotes within
+       # quotes we have to do it in 2 steps:
        extractedcf=`$SED -n -e "$sed_extractcf" < "$progpath"`
        eval "$extractedcf"
       else
@@ -610,7 +610,7 @@ Otherwise, only FILE itself is deleted using RM."
                        set -x
                        ;;
 
-      --dlopen)                test $# -eq 0 && func_missing_arg "$opt" && break
+      --dlopen)                test "$#" -eq 0 && func_missing_arg "$opt" && break
                        execute_dlfiles="$execute_dlfiles $1"
                        ;;
 
@@ -618,7 +618,7 @@ Otherwise, only FILE itself is deleted using RM."
       --features)       func_features                                  ;;
       --finish)                mode="finish"                                   ;;
 
-      --mode)          test $# -eq 0 && func_missing_arg "$opt" && break
+      --mode)          test "$#" -eq 0 && func_missing_arg "$opt" && break
                        case $1 in
                          # Valid mode arguments:
                          clean)        ;;
@@ -648,7 +648,7 @@ Otherwise, only FILE itself is deleted using RM."
                        show=:
                        ;;
 
-      --tag)           test $# -eq 0 && func_missing_arg "$opt" && break
+      --tag)           test "$#" -eq 0 && func_missing_arg "$opt" && break
                        preserve_args="$preserve_args $opt $1"
                        func_enable_tag "$1"    # tagname is set here
                        shift