tests: do not uselessly require GNU make or gcc in a few tests

* tests/lex3.test: Remove gcc from requirements, as any working C
compiler should be ok.  Consequently, do not clobber user CFLAGS.
Also, remove GNUmake from requirements; it was added (see commit
`Release-1-8-103-g0d2f592') because this test fails with FreeBSD
make due to VPATH issues -- but so do many other yacc-related and
lex-related tests currently, and requiring GNU make in all of
them would unacceptably reduce coverage.
* tests/lexvpath.test: Remove gcc from requirements, as any
working C compiler should be ok.
* tests/yacc4.test: Likewise.
* tests/yacc8.test: Likewise.
* tests/lex5.test: Likewise.  Also, do not require anymore GNU
make; to compensate, explicitly call "$MAKE Makefile" to update
the out-of-date Makefile if $MAKE is not GNU make.

diff --git a/ChangeLog b/ChangeLog
index c0495d5ada2adbcb66e7a73f1c2cbabc1018d741..09511ff4b2b135dbcb6b69720844c16c44cd4f34 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,21 @@
+2011-04-11  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       tests: do not uselessly require GNU make or gcc in a few tests
+       * tests/lex3.test: Remove gcc from requirements, as any working C
+       compiler should be ok.  Consequently, do not clobber user CFLAGS.
+       Also, remove GNUmake from requirements; it was added (see commit
+       `Release-1-8-103-g0d2f592') because this test fails with FreeBSD
+       make due to VPATH issues -- but so do many other yacc-related and
+       lex-related tests currently, and requiring GNU make in all of
+       them would unacceptably reduce coverage.
+       * tests/lexvpath.test: Remove gcc from requirements, as any
+       working C compiler should be ok.
+       * tests/yacc4.test: Likewise.
+       * tests/yacc8.test: Likewise.
+       * tests/lex5.test: Likewise.  Also, do not require anymore GNU
+       make; to compensate, explicitly call "$MAKE Makefile" to update
+       the out-of-date Makefile if $MAKE is not GNU make.
+
 2011-04-10  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
        coverage: test mixed C/C++ yacc-generated parsers in the same dir

diff --git a/tests/lex3.test b/tests/lex3.test
index 536239b2eb072a3e33eb831897b818c8e4e7d00c..7509d521f0a08ab05c96d725fcd83654b214d49d 100755 (executable)
--- a/tests/lex3.test
+++ b/tests/lex3.test
@@ -19,15 +19,11 @@
 # Test associated with PR 19.
 # From Matthew D. Langston.
 
-required='gcc lex GNUmake'
+required=lex
 . ./defs || Exit 1
 
 set -e
 
-# Ignore user CFLAGS.
-CFLAGS=
-export CFLAGS
-
 distdir=$me-1.0
 
 cat >> configure.in << 'END'

diff --git a/tests/lex5.test b/tests/lex5.test
index f3ddcccf6008b95c855b4c2fd51a152738441c25..12d9f00277023d09d15b389b42f5986b24472d13 100755 (executable)
--- a/tests/lex5.test
+++ b/tests/lex5.test
@@ -17,7 +17,7 @@
 
 # Test for subdir lexers.
 
-required='gcc GNUmake lex'
+required=lex
 . ./defs || Exit 1
 
 set -e
@@ -87,6 +87,7 @@ $AUTOMAKE -a --no-force
 test -f ./ylwrap
 
 cd sub
+using_gnumake || $MAKE Makefile
 $MAKE foo/foo2.o
 test -f foo/foo2.c
 test -f foo/foo2.o

diff --git a/tests/lexvpath.test b/tests/lexvpath.test
index 54a517a2c6328c743d751f33ee1fd51276ad5e11..12f9ca6a8e1cd9ad0b1a8ce0c4a5946016f16c7f 100755 (executable)
--- a/tests/lexvpath.test
+++ b/tests/lexvpath.test
@@ -22,7 +22,7 @@
 
 # Please keep this in sync with sister test `yaccvapth.test'.
 
-required='gcc lex'
+required=lex
 . ./defs || Exit 1
 
 set -e

diff --git a/tests/yacc4.test b/tests/yacc4.test
index 1d508edc08ed843b1413d321349aeabb85eaa961..87a5157020b25cee3f2d6ced1cb8266ac8629661 100755 (executable)
--- a/tests/yacc4.test
+++ b/tests/yacc4.test
@@ -16,7 +16,7 @@
 
 # Some simple tests of ylwrap functionality.
 
-required='yacc gcc'
+required=yacc
 . ./defs || Exit 1
 
 set -e
@@ -36,7 +36,7 @@ END
 # First parser.
 cat > parse.y << 'END'
 %{
-int yylex () {return 0;}
+int yylex () { return 0; }
 void yyerror (char *s) {}
 %}
 %%
@@ -46,7 +46,7 @@ END
 # Second parser.
 cat > bar.y << 'END'
 %{
-int yylex () {return 0;}
+int yylex () { return 0; }
 void yyerror (char *s) {}
 %}
 %%

diff --git a/tests/yacc8.test b/tests/yacc8.test
index ca1ac686ca6b449f6c8395fefe9e453b6d2b3a16..158e64ffda9a6f3338f0d7777e50a82c8203afc8 100755 (executable)
--- a/tests/yacc8.test
+++ b/tests/yacc8.test
@@ -17,7 +17,7 @@
 
 # Test for subdir parsers.
 
-required="gcc yacc"
+required=yacc
 
 . ./defs || Exit 1