[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: CVS automake testsuite failures on NetBSD i386 2.0F
From: |
Alexandre Duret-Lutz |
Subject: |
Re: CVS automake testsuite failures on NetBSD i386 2.0F |
Date: |
Mon, 07 Jun 2004 08:04:26 +0200 |
User-agent: |
Gnus/5.1003 (Gnus v5.10.3) Emacs/21.3.50 (gnu/linux) |
>>> "Nicolas" == Nicolas Joly <address@hidden> writes:
[...]
Nicolas> With Tru64 unix and Solaris `/bin/sh', things look worse :
Nicolas> $ for i in a b c; do echo $i; done
Nicolas> a
Nicolas> b
Nicolas> c
Nicolas> $ for i in ${x-a b c}; do echo $i; done
Nicolas> bad substitution
Nicolas> $ x='x y'
Nicolas> $ for i in ${x-a b c}; do echo $i; done
Nicolas> bad substitution
Maybe that syntax is truly incorrect? I'm unable to tell.
POSIX refers to the right hand side of `-' as "word".
I'm installing the appended patch.
Nicolas> even zsh does not seems to behave very well :
Zsh never performs field splitting after parameter expansion by
default. You have to `setopt shwordsplit' or `emulate sh' for
things will work (configure does the latter).
2004-06-07 Alexandre Duret-Lutz <address@hidden>
* m4/tar.m4 (_AM_PROG_TAR): Split the definition of $_am_tools
so it works with Solaris and Tru64 /bin/sh.
Report from Nicolas Joly.
Index: m4/tar.m4
===================================================================
RCS file: /cvs/automake/automake/m4/tar.m4,v
retrieving revision 1.3
diff -u -r1.3 tar.m4
--- m4/tar.m4 6 Jun 2004 20:42:14 -0000 1.3
+++ m4/tar.m4 7 Jun 2004 06:00:38 -0000
@@ -42,9 +42,10 @@
[m4_fatal([Unknown tar format])])
AC_MSG_CHECKING([how to create a $1 tar archive])
# Loop over all known methods to create a tar archive until one works.
-_am_tools=${am_cv_prog_tar_$1-gnutar m4_if([$1], [ustar], [plaintar]) pax cpio
none}
-# Do not replace $_am_tools by its definition on the `for' line, NetBSD 2.0F
-# /bin/sh will not perform IFS splitting on the right side of `-'.
+_am_tools='gnutar m4_if([$1], [ustar], [plaintar]) pax cpio none'
+_am_tools=${am_cv_prog_tar_$1-$_am_tools}
+# Do not fold the above two lines into one, because Tru64 sh and
+# Solaris sh will not grok spaces in the rhs of `-'.
for _am_tool in $_am_tools
do
case $_am_tool in
--
Alexandre Duret-Lutz