automake-patches
[Top][All Lists]
Advanced

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

[FYI] {master} tests: fix spurious failure in 'color2.test'


From: Stefano Lattarini
Subject: [FYI] {master} tests: fix spurious failure in 'color2.test'
Date: Tue, 3 May 2011 10:00:12 +0200
User-agent: KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; )

I've caught this testsuite bug by running the testsuite with
MAKE="make -j2".  The fix I've checked in might not be optimal,
but it works Right Now, and I don't now enough Expect/TCL to
find out if there's a better fix.  Any suggestion in this
regard would be appreciated.

Regards,
  Stefano

-*-*-

tests: fix spurious failure in 'color2.test'

* tests/color2.test: If $MAKE contains command-line arguments (as
in e.g., "make -j2"), expect's directive "spawn $env(MAKE)" fails
spuriously, because it tries to run "$MAKE" as a single command.
Fix this with proper uses of the TCL `eval' builtin.
---
 ChangeLog         |    8 ++++++++
 tests/color2.test |    4 ++--
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 046ecff..a5354c1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2011-05-03  Stefano Lattarini  <address@hidden>
+
+       tests: fix spurious failure in 'color2.test'
+       * tests/color2.test: If $MAKE contains command-line arguments (as
+       in e.g., "make -j2"), expect's directive "spawn $env(MAKE)" fails
+       spuriously, because it tries to run "$MAKE" as a single command.
+       Fix this with proper uses of the TCL `eval' builtin.
+
 2011-05-01  Stefano Lattarini  <address@hidden>
 
        tests: fix self check spurious failure with older bash versions
diff --git a/tests/color2.test b/tests/color2.test
index c3315de..eb9c527 100755
--- a/tests/color2.test
+++ b/tests/color2.test
@@ -39,7 +39,7 @@ esac
 # Check that we have a working expect program.
 cat >expect-check <<'END'
 #! /usr/bin/expect -f
-spawn $env(THE_SYSTEM_SHELL) -c :
+eval spawn $env(THE_SYSTEM_SHELL) -c :
 expect eof
 END
 THE_SYSTEM_SHELL=/bin/sh expect -f expect-check || {
@@ -107,7 +107,7 @@ test_no_color ()
 
 cat >expect-make <<'END'
 #! /usr/bin/expect -f
-spawn $env(MAKE) -e check
+eval spawn $env(MAKE) -e check
 expect eof
 END
 
-- 
1.7.2.3



reply via email to

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