bug-automake
[Top][All Lists]
Advanced

[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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]