testsuite: require C++ compiler explicitly in tests needing it

The list of the test scripts which needed a C++ compiler but
failed to require it explicitly has been found by running:
  $ tests=`grep -lE 'CXX|\.c(c|\+\+|xx|pp)' *.test | tr '\n' ' '`
  $ am_explicit_skips=yes CXX=false make check TESTS="$tests"
and looking for tests that reported FAIL instead of SKIP.
After the present change, all those previously failing tests
either pass or get correctly skipped.

* tests/subobj9.test: Require `c++', instead of explicitly
skipping on configure failure.  Add excerpts and/or details from
the original bug report that prompted this tests to be written.
* tests/silentcxx-gcc.test: Require g++.
* tests/silentcxx.test: Require c++.
* tests/suffix3.test: Likewise.

diff --git a/ChangeLog b/ChangeLog
index e7f480ff8d7d576c2561c781cdb0ba43c55ea8d2..d26268d15cb0d1a1a61429c0438a84a8c15bad66 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,20 @@
+2011-05-25  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       testsuite: require C++ compiler explicitly in tests needing it
+       The list of the test scripts which needed a C++ compiler but
+       failed to require it explicitly has been found by running:
+         $ tests=`grep -lE 'CXX|\.c(c|\+\+|xx|pp)' *.test | tr '\n' ' '`
+         $ am_explicit_skips=yes CXX=false make check TESTS="$tests"
+       and looking for tests that reported FAIL instead of SKIP.
+       After the present change, all those previously failing tests
+       either pass or get correctly skipped.
+       * tests/subobj9.test: Require `c++', instead of explicitly
+       skipping on configure failure.  Add excerpts and/or details from
+       the original bug report that prompted this tests to be written.
+       * tests/silentcxx-gcc.test: Require g++.
+       * tests/silentcxx.test: Require c++.
+       * tests/suffix3.test: Likewise.
+
 2011-05-25  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
        testsuite: don't require g++ where any C++ compiler is enough

diff --git a/tests/silentcxx-gcc.test b/tests/silentcxx-gcc.test
index 1f5768c2d025e6e09cf447f38b981f2c4f5697dd..55f296d0abc5df50e01e39db7a0b95a4ac6d0188 100755 (executable)
--- a/tests/silentcxx-gcc.test
+++ b/tests/silentcxx-gcc.test
@@ -18,6 +18,7 @@
 # This test requires the GNU C++ compiler; keep it in sync with sister
 # test `silentcxx.test', which should work with generic compilers.
 
+required=g++
 . ./defs || Exit 1
 
 mkdir sub

diff --git a/tests/silentcxx.test b/tests/silentcxx.test
index b4919c775569e35a7bae19f3105d2d12364dc2e6..14825db24c780f000f39b08bb145a8be38b5e0aa 100755 (executable)
--- a/tests/silentcxx.test
+++ b/tests/silentcxx.test
@@ -19,6 +19,7 @@
 # sister test `silentcxx-gcc.test', which requires the GNU C++ compiler
 # and forces the use of gcc depmode.
 
+required=c++
 . ./defs || Exit 1
 
 mkdir sub

diff --git a/tests/subobj9.test b/tests/subobj9.test
index f0e90071421d26367e88a30aa9d2c078aa603a60..9bab174a147bef9a45ac08f8b3195f65db776238 100755 (executable)
--- a/tests/subobj9.test
+++ b/tests/subobj9.test
@@ -16,8 +16,14 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 # Test for PR 312.
+#
+# == Report ==
+# When using non-recursive make to build a libtoolize-library from
+# sources in a subdirectory, 'make distcheck' fails because of incomplete
+# cleanup. "make clean" tries to remove `*.o' and `.../<file>.lo' but
+# forgets `.../<file>.o'.
 
-required=libtoolize
+required='c++ libtoolize'
 . ./defs || Exit 1
 
 cat > configure.in << END
@@ -60,8 +66,7 @@ $ACLOCAL
 $AUTOCONF
 $AUTOMAKE -a
 
-# Skip this test on configure errors (e.g., broken C++ compilers).
-./configure || skip_ "configure failure"
+./configure
 
 # Ensure './libtool --help' will use the right tool versions.
 export AUTOCONF AUTOMAKE

diff --git a/tests/suffix3.test b/tests/suffix3.test
index e0c01f618de7333a6a4453f308b60c1b9f9ba591..70385feae5fd4aaddc0258414b2ff748ba0c7c2e 100755 (executable)
--- a/tests/suffix3.test
+++ b/tests/suffix3.test
@@ -17,6 +17,7 @@
 
 # Test to make sure that suffix rules chain.
 
+required=c++
 . ./defs || Exit 1
 
 cat >> configure.in << 'END'