FYI: libtool--devo--1.0--patch-180

[Gary V. Vaughan]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Applied to HEAD.
- -- 
Gary V. Vaughan      ())_.  address@hidden,gnu.org}
Research Scientist   ( '/   http://tkd.kicks-ass.net
GNU Hacker           / )=   http://www.gnu.org/software/libtool
Technical Author   `(_~)_   http://sources.redhat.com/autobook
_________________________________________________________
This patch notification generated by tlaapply version 0.5
http://tkd.kicks-ass.net/arch/address@hidden/cvs-utils--tla--1.0
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2 (Darwin)

iD8DBQFBR3UOFRMICSmD1gYRAkeeAKDA+IH92L/zamPdCuNJcWN6ZY8/ZQCgyNnA
TJHvDTM9mdNwfkLKSAME1p4=
=Ox7c
-----END PGP SIGNATURE-----

* looking for address@hidden/libtool--devo--1.0--patch-179 to compare with
* comparing to address@hidden/libtool--devo--1.0--patch-179
M  ChangeLog
M  config/ltmain.in

* modified files

Index: Changelog
from  Gary V. Vaughan  <address@hidden>

        * config/ltmain.in (func_quote_for_eval): Factor out the ugly
        shell meta-character quoting.  Changed all callers.

--- orig/config/ltmain.in
+++ mod/config/ltmain.in
@@ -739,6 +739,25 @@
 }
 
 
+# func_quote_for_eval arg
+# Aesthetically quote ARG to be evaled later.
+func_quote_for_eval ()
+{
+  my_arg="$1"
+
+  case $my_arg in
+    # Double-quote args containing other shell metacharacters.
+    # Many Bourne shells cannot handle close brackets correctly
+    # in scan sets, so we specify it separately.
+    *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \      ]*|*]*|"")
+      my_arg="\"$my_arg\""
+      ;;
+  esac
+
+  echo "$my_arg"
+}
+
+
 # func_check_version_match
 # Ensure that we are using m4 macros, and libtool script from the same
 # release of libtool.
@@ -825,12 +844,7 @@
     if test -n "$available_tags" && test -z "$tagname"; then
       CC_quoted=
       for arg in $CC; do
-       case $arg in
-         *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \ ]*|*]*|"")
-         arg="\"$arg\""
-         ;;
-       esac
-       CC_quoted="$CC_quoted $arg"
+       CC_quoted="$CC_quoted "`func_quote_for_eval "$arg"`
       done
       case $@ in
       # Blanks in the command may have been stripped by the calling shell,
@@ -845,14 +859,9 @@
            eval "`${SED} -n -e '/^# ### BEGIN LIBTOOL TAG CONFIG: '$z'$/,/^# 
### END LIBTOOL TAG CONFIG: '$z'$/p' < $progpath`"
            CC_quoted=
            for arg in $CC; do
-           # Double-quote args containing other shell metacharacters.
-           case $arg in
-             *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \     ]*|*]*|"")
-             arg="\"$arg\""
-             ;;
-           esac
-           CC_quoted="$CC_quoted $arg"
-         done
+             # Double-quote args containing other shell metacharacters.
+             CC_quoted="$CC_quoted "`func_quote_for_eval "$arg"`
+           done
            case "$@ " in
              " $CC "* | "$CC "* | " `$echo $CC` "* | "`$echo $CC` "* | " 
$CC_quoted"* | "$CC_quoted "* | " `$echo $CC_quoted` "* | "`$echo $CC_quoted` 
"*)
              # The compiler in the base compile command matches
@@ -1233,16 +1242,7 @@
          save_ifs="$IFS"; IFS=','
          for arg in $args; do
            IFS="$save_ifs"
-
-           # Double-quote args containing other shell metacharacters.
-           # Many Bourne shells cannot handle close brackets correctly
-           # in scan sets, so we specify it separately.
-           case $arg in
-             *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \     ]*|*]*|"")
-             arg="\"$arg\""
-             ;;
-           esac
-           lastarg="$lastarg $arg"
+           lastarg="$lastarg "`func_quote_for_eval "$arg"`
          done
          IFS="$save_ifs"
          lastarg=`$echo "X$lastarg" | $Xsed -e "s/^ //"`
@@ -1265,17 +1265,7 @@
 
       # Aesthetically quote the previous argument.
       lastarg=`$echo "X$lastarg" | $Xsed -e "$sed_quote_subst"`
-
-      case $lastarg in
-      # Double-quote args containing other shell metacharacters.
-      # Many Bourne shells cannot handle close brackets correctly
-      # in scan sets, so we specify it separately.
-      *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \    ]*|*]*|"")
-       lastarg="\"$lastarg\""
-       ;;
-      esac
-
-      base_compile="$base_compile $lastarg"
+      base_compile="$base_compile "`func_quote_for_eval "$lastarg"`
     done # for arg
 
     case $arg_mode in
@@ -1812,12 +1802,7 @@
        $echo "X$nonopt" | $Xsed | $GREP shtool > /dev/null; then
       # Aesthetically quote it.
       arg=`$echo "X$nonopt" | $Xsed -e "$sed_quote_subst"`
-      case $arg in
-      *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \    ]*|*]*)
-       arg="\"$arg\""
-       ;;
-      esac
-      install_prog="$arg "
+      install_prog=`func_quote_for_eval "$arg "`
       arg="$1"
       shift
     else
@@ -1828,12 +1813,7 @@
     # The real first argument should be the name of the installation program.
     # Aesthetically quote it.
     arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
-    case $arg in
-    *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \      ]*|*]*)
-      arg="\"$arg\""
-      ;;
-    esac
-    install_prog="$install_prog$arg"
+    install_prog="$install_prog"`func_quote_for_eval "$arg"`
 
     # We need to accept at least all the BSD install flags.
     dest=
@@ -1876,12 +1856,7 @@
 
       # Aesthetically quote the argument.
       arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
-      case $arg in
-      *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \    ]*|*]*)
-       arg="\"$arg\""
-       ;;
-      esac
-      install_prog="$install_prog $arg"
+      install_prog="$install_prog "`func_quote_for_eval "$arg"`
     done
 
     test -z "$install_prog" && \
@@ -2393,15 +2368,9 @@
 
     # Go through the arguments, transforming them on the way.
     while test "$#" -gt 0; do
-      arg="$1"
+      arg=`$echo "X$1" | $Xsed -e "$sed_quote_subst"`
+      libtool_args="$libtool_args "`func_quote_for_eval "$arg"`
       shift
-      case $arg in
-      *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \    ]*|*]*|"")
-       qarg=\"`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`\" ### testsuite: 
skip nested quoting test
-       ;;
-      *) qarg=$arg ;;
-      esac
-      libtool_args="$libtool_args $qarg"
 
       # If the previous option needs an argument, assign it.
       if test -n "$prev"; then
@@ -2917,11 +2886,7 @@
        save_ifs="$IFS"; IFS=','
        for flag in $args; do
          IFS="$save_ifs"
-         case $flag in
-           *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \       ]*|*]*|"")
-           flag="\"$flag\""
-           ;;
-         esac
+         flag=`func_quote_for_eval "$flag"`
          arg="$arg $wl$flag"
          compiler_flags="$compiler_flags $flag"
        done
@@ -2935,11 +2900,7 @@
        save_ifs="$IFS"; IFS=','
        for flag in $args; do
          IFS="$save_ifs"
-         case $flag in
-           *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \       ]*|*]*|"")
-           flag="\"$flag\""
-           ;;
-         esac
+         flag=`func_quote_for_eval "$flag"`
          arg="$arg $wl$flag"
          compiler_flags="$compiler_flags $wl$flag"
          linker_flags="$linker_flags $flag"
@@ -2968,31 +2929,19 @@
       # +DA*, +DD* enable 64-bit mode on the HP compiler
       # -q* pass through compiler args for the IBM compiler
       # -m* pass through architecture-specific compiler args for GCC
-      -64|-mips[0-9]||-xarch=*|-xtarget=*|+DA*|+DD*|-q*|-m*)
-       # Unknown arguments in both finalize_command and compile_command need
-       # to be aesthetically quoted because they are evaled later.
-       arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
-       case $arg in
-       *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \   ]*|*]*|"")
-       arg="\"$arg\""
-       ;;
-      esac
-      compile_command="$compile_command $arg"
-      finalize_command="$finalize_command $arg"
-      compiler_flags="$compiler_flags $arg"
-      continue
-      ;;
+      -64|-mips[0-9]|-xarch=*|-xtarget=*|+DA*|+DD*|-q*|-m*)
+        arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
+       arg=`func_quote_for_eval "$arg"`
+        compile_command="$compile_command $arg"
+        finalize_command="$finalize_command $arg"
+        compiler_flags="$compiler_flags $arg"
+        continue
+        ;;
 
       # Some other compiler flag.
       -* | +*)
-       # Unknown arguments in both finalize_command and compile_command need
-       # to be aesthetically quoted because they are evaled later.
-       arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
-       case $arg in
-       *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \   ]*|*]*|"")
-         arg="\"$arg\""
-         ;;
-       esac
+        arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
+        arg=`func_quote_for_eval "$arg"`
        ;;
 
       *.$objext)
@@ -3125,11 +3074,7 @@
        # Unknown arguments in both finalize_command and compile_command need
        # to be aesthetically quoted because they are evaled later.
        arg=`$echo "X$arg" | $Xsed -e "$sed_quote_subst"`
-       case $arg in
-       *[\[\~\#\^\&\*\(\)\{\}\|\;\<\>\?\'\ \   ]*|*]*|"")
-         arg="\"$arg\""
-         ;;
-       esac
+       arg=`func_quote_for_eval "$arg"`
        ;;
       esac # arg