* TODO: Make a note to undo this later.

From Christoph Pfisterer <cp@chrisp.de>
* libtool.m4: Use $() execution substitution quotation to work
around zsh builtin echo  escape removal.

diff --git a/ChangeLog b/ChangeLog
index 4884390fee95d14d116396c0288359074868936d..d05a6ccc9216f5b1d06e92c6de8ccff48af52a6f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2001-04-23  Gary V. Vaughan  <gvv@techie.com>
+
+       * TODO: Make a note to undo this later.
+       From Christoph Pfisterer <cp@chrisp.de>
+       * libtool.m4: Use $() execution substitution quotation to work
+       around zsh builtin echo \ escape removal.
+
 2001-04-22  Peter Eisentraut  <peter_e@gmx.net>
 
        * bootstrap: Allow overriding the location aclocal, automake,

diff --git a/TODO b/TODO
index 9d6121aa1cbc794c1218489ddbf7cd356cd3d470..2262c58878bc0a8665b1ee7e13997263c287adf6 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,6 +1,9 @@
 In the near future:
 ********************
 
+* Figure out why zsh echo builtin's removal of \ escapes is not detected
+  by libtool's echo selector.
+
 * Port the migration of all code from ltconfig into libtool.m4 to the
 multi-language-branch, so that CVS automake can remove its references
 to ltconfig.

diff --git a/libtool.m4 b/libtool.m4
index 1c7b6a85e5aa82e330eac2daa647a0da3c996244..80e51fbf7c7a4e8f3b7e9fa1e1c48d883474cb50 100644 (file)
--- a/libtool.m4
+++ b/libtool.m4
@@ -1506,7 +1506,10 @@ else
 
   darwin* | rhapsody*)
     allow_undefined_flag='-undefined suppress'
-    archive_cmds='$CC `test .$module = .yes && echo -bundle || echo -dynamiclib` $allow_undefined_flag -o $lib $libobjs $deplibs$linkopts -install_name $rpath/$soname `test -n "$verstring" -a x$verstring != x0.0 && echo $verstring`'
+    # FIXME: Relying on posixy $() will cause problems for
+    #        cross-compilation, but unfortunately the echo tests do not
+    #        yet detect zsh echo's removal of \ escapes.
+    archive_cmds='$CC $(test .$module = .yes && echo -bundle || echo -dynamiclib) $allow_undefined_flag -o $lib $libobjs $deplibs$linkopts -install_name $rpath/$soname $(test -n "$verstring" -a x$verstring != x0.0 && echo $verstring)'
     # We need to add '_' to the symbols in $export_symbols first
     #archive_expsym_cmds="$archive_cmds"' && strip -s $export_symbols'
     hardcode_direct=yes
@@ -1928,8 +1931,11 @@ darwin* | rhapsody*)
   version_type=darwin
   need_lib_prefix=no
   need_version=no
-  library_names_spec='${libname}${release}${versuffix}.`test .$module = .yes && echo so || echo dylib` ${libname}${release}${major}.`test .$module = .yes && echo so || echo dylib` ${libname}.`test .$module = .yes && echo so || echo dylib`'
-  soname_spec='${libname}${release}${major}.`test .$module = .yes && echo so || echo dylib`'
+  # FIXME: Relying on posixy $() will cause problems for
+  #        cross-compilation, but unfortunately the echo tests do not
+  #        yet detect zsh echo's removal of \ escapes.
+  library_names_spec='${libname}${release}${versuffix}.$(test .$module = .yes && echo so || echo dylib) ${libname}${release}${major}.$(test .$module = .yes && echo so || echo dylib) ${libname}.$(test .$module = .yes && echo so || echo dylib)'
+  soname_spec='${libname}${release}${major}.$(test .$module = .yes && echo so || echo dylib)'
   shlibpath_overrides_runpath=yes
   shlibpath_var=DYLD_LIBRARY_PATH
   ;;