From: Stefano Lattarini <stefano.lattarini@gmail.com>
Date: Fri, 13 May 2011 13:03:20 +0000 (+0200)
Subject: lex tests: remove erroneous check about ylwrap distribution
X-Git-Tag: ylwrap-refactor-abandoned-branch~17^2~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=68da6ae8ce04258668e63aee320011c8bf130d18;p=thirdparty%2Fautomake.git

lex tests: remove erroneous check about ylwrap distribution

* tests/lex5.test: Do not check that the ylwrap script is *not*
distributed when there is only one lexer, as ylwrap is in fact
expected to distributed unconditionally (at least starting from
commit `Release-1-9-323-gc5881cc' of 19-08-2006).  Note that this
bogus check wasn't triggering any failure because it was done at
the wrong time (i.e. from the VPATH build directory), so that it
could never fail anyway.
Instead, check that ylwrap is distributed even when there is only
one lexer.
Since we are at it, add a couple of `ls -l' calls, to get better
debugging info.
---

diff --git a/ChangeLog b/ChangeLog
index 7716923ce..57916e79f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2011-05-13  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+	lex tests: remove erroneous check about ylwrap distribution
+	* tests/lex5.test: Do not check that the ylwrap script is *not*
+	distributed when there is only one lexer, as ylwrap is in fact
+	expected to distributed unconditionally (at least starting from
+	commit `Release-1-9-323-gc5881cc' of 19-08-2006).  Note that this
+	bogus check wasn't triggering any failure because it was done at
+	the wrong time (i.e. from the VPATH build directory), so that it
+	could never fail anyway.
+	Instead, check that ylwrap is distributed even when there is only
+	one lexer.
+	Since we are at it, add a couple of `ls -l' calls, to get better
+	debugging info.
+
 2011-05-13  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
 	yacc tests: "make clean" removes C++ files from non-distributed .y
diff --git a/tests/lex5.test b/tests/lex5.test
index 12d9f0027..fd394eb57 100755
--- a/tests/lex5.test
+++ b/tests/lex5.test
@@ -58,15 +58,18 @@ $ACLOCAL
 $AUTOCONF
 $AUTOMAKE -a
 
+# We expect ylwrap to be used and distributed even if there is
+# only one lexer.
+test -f ylwrap
+
 mkdir sub
 cd sub
 ../configure
 $MAKE foo/foo.o
+ls -l # For debugging.
 
 test -f foo/foo.c
 test -f foo/foo.o
-# ylwrap is not needed
-test ! -f ./ylwrap
 
 # Now, adds another lexer to test ylwrap.
 
@@ -84,11 +87,11 @@ END
 $sleep
 
 $AUTOMAKE -a --no-force
-test -f ./ylwrap
 
 cd sub
 using_gnumake || $MAKE Makefile
 $MAKE foo/foo2.o
+ls -l # For debugging.
 test -f foo/foo2.c
 test -f foo/foo2.o