[Top][All Lists]
[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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [FYI] {master} tests: fix spurious failure in 'color2.test',
Stefano Lattarini <=