automake: if TEST_EXTENSIONS is set to empty, don't look inside it.

This change fixes https://bugs.gnu.org/42635.

* bin/automake.in (handle_tests): do not use $test_suffixes[0]
if it does not exist.
* t/test-extensions-empty.sh: new test.
* t/list-of-tests.mk (handwritten_TESTS): add it.

diff --git a/bin/automake.in b/bin/automake.in
index 67f6f3a6784c007f699cf4c49dba0aae13b2d9fe..c12078711b04de6b5e22d9efb99669c0d2a0f8c9 100644 (file)
--- a/bin/automake.in
+++ b/bin/automake.in
@@ -4954,7 +4954,7 @@ sub handle_tests ()
          if ($handle_exeext)
            {
              unshift (@test_suffixes, $at_exeext)
-               unless $test_suffixes[0] eq $at_exeext;
+               unless @test_suffixes && $test_suffixes[0] eq $at_exeext;
            }
          unshift (@test_suffixes, '');
 

diff --git a/t/list-of-tests.mk b/t/list-of-tests.mk
index 16083137d069db84b015c35ebdb614d701e2c910..3bd1211105c269672ffe008878b63284a1c9f03d 100644 (file)
--- a/t/list-of-tests.mk
+++ b/t/list-of-tests.mk
@@ -832,6 +832,7 @@ t/test-driver-strip-vpath.sh \
 t/test-driver-trs-suffix-registered.sh \
 t/test-driver-fail.sh \
 t/test-driver-is-distributed.sh \
+t/test-extensions-empty.sh \
 t/test-harness-vpath-rewrite.sh \
 t/test-log.sh \
 t/test-logs-repeated.sh \

diff --git a/t/test-extensions-empty.sh b/t/test-extensions-empty.sh
new file mode 100755 (executable)
index 0000000..5f466e0
--- /dev/null
+++ b/t/test-extensions-empty.sh
@@ -0,0 +1,52 @@
+#! /bin/sh
+# Copyright (C) 2020 Free Software Foundation, Inc.
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2, or (at your option)
+# any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <https://www.gnu.org/licenses/>.
+
+# Empty assignment to TEST_EXTENSIONS should not provoke Perl warning.
+# https://bugs.gnu.org/42635
+
+. test-init.sh
+
+cat > configure.ac << 'END'
+AC_INIT([foo],[1.0])
+AM_INIT_AUTOMAKE([foreign])
+AC_PROG_CC  dnl comment this line to make the warning disappear
+AC_CONFIG_FILES([Makefile])
+AC_OUTPUT
+END
+
+cat > Makefile.am << 'END'
+TEST_EXTENSIONS =
+LOG_COMPILER = echo
+TESTS = foo.test
+END
+
+touch foo.test
+
+autoreconf -fi >reconf.out 2>&1 
+grep 'uninitialized value' reconf.out && exit 1
+
+# What we're trying to avoid:
+# ...
+# Use of uninitialized value in string eq at /usr/bin/automake line 4953.
+# ...
+# nl -ba `command -v automake` | sed -n '4951,4955p'
+#  4951            if ($handle_exeext)
+#  4952              {
+#  4953                unshift (@test_suffixes, $at_exeext)
+#  4954                  unless $test_suffixes[0] eq $at_exeext;
+#  4955              }
+
+: