automake-patches
[Top][All Lists]
Advanced

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

[FYI] {master} tests: avoid spurious failure on NetBSD


From: Stefano Lattarini
Subject: [FYI] {master} tests: avoid spurious failure on NetBSD
Date: Mon, 13 Feb 2012 22:26:28 +0100

* tests/parallel-tests-fd-redirect-exeext.test (configure.in): Tweak
so that we don't end up with the generated configure script having
two nested subshells that start with '((':

  ((./conftest$EXEEXT 9>&1) | grep "^foobar") >&5 2>&5

which is unportable and could confuse some shells (e.g., NetBSD 5.1
/bin/ksh) into thinking we are trying to perform an arithmetic
operation.
---
 tests/parallel-tests-fd-redirect-exeext.test |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/tests/parallel-tests-fd-redirect-exeext.test 
b/tests/parallel-tests-fd-redirect-exeext.test
index 0f9d6da..4426604 100755
--- a/tests/parallel-tests-fd-redirect-exeext.test
+++ b/tests/parallel-tests-fd-redirect-exeext.test
@@ -34,7 +34,11 @@ AC_LINK_IFELSE(
     [AC_LANG_PROGRAM([[#include <unistd.h>]],
                      [[write (9, "foobar\n", 7); return 0;]])],
     [AM_RUN_LOG([./conftest$EXEEXT 9>&1]) \
-       && AM_RUN_LOG([(./conftest$EXEEXT 9>&1) | grep "^foobar"]) \
+dnl Leading ":;" required to avoid having two nested subshells starting
+dnl with '((' in the generated configure: that is unportable and could
+dnl confuse some shells (e.g., NetBSD 5.1 /bin/ksh) into thinking we are
+dnl trying to perform an arithmetic operation.
+       && AM_RUN_LOG([:; (./conftest$EXEEXT 9>&1) | grep "^foobar"]) \
        && am__ok=yes])
 test $am__ok = yes || AS_EXIT([63])
 AM_CONDITIONAL([real_EXEEXT], [test -n "$EXEEXT"])
-- 
1.7.7.3




reply via email to

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