automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[FYI] {maint} tests: work around a ksh bug w.r.t.


From: Stefano Lattarini
Subject: [FYI] {maint} tests: work around a ksh bug w.r.t.
Date: Sat, 4 Aug 2012 17:43:12 +0200

Fixes automake bug#10898.  See also the older (much older) thread:
<http://lists.gnu.org/archive/html/automake-patches/2009-12/msg00036.html>

At least the AT&T and OpenSolaris versions of the Korn shell, as well
as the /bin/sh from OpenIndiana 11, have a strange bug regarding the
expansion of ${1+"$@"}: when exactly *one empty* argument is passed to
a script run by one of this shells, inside that script ${1+"$@"} will
expand to *nothing*, rather than to to the single empty string, as
one would expect (OTOH, $# will correctly expand to 1).  This buggy
behaviour was causing a spurious failure in our testsuite (test 6 in
't/automake-cmdline.tap').  Work around it.

* t/wrap/automake.in: Here.

Signed-off-by: Stefano Lattarini <address@hidden>
---
 t/wrap/automake.in | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/t/wrap/automake.in b/t/wrap/automake.in
index 14f437f..b77177c 100644
--- a/t/wrap/automake.in
+++ b/t/wrap/automake.in
@@ -27,4 +27,10 @@ fi
 
 perllibdir="@abs_top_builddir@/address@hidden@@abs_top_srcdir@/lib"
 export perllibdir
-exec "@abs_top_builddir@/automake" "address@hidden@/lib" ${1+"$@"}
+# Don't trust ${1+"$@"}: it has subtle bugs in some Korn shells (for very
+# corner case only admittedly, but those have already bitten us -- see
+# automake bug#10898).
+case $# in
+  0) exec "@abs_top_builddir@/automake" "address@hidden@/lib";;
+  *) exec "@abs_top_builddir@/automake" "address@hidden@/lib" "$@";;
+esac
-- 
1.7.12.rc0




reply via email to

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