* Makefile.am (maintainer-check): Make sure `$MAKE -e' is

always followed by a SHELL setting.
* tests/ansi3.test, tests/ansi5.test, tests/subobj3.test: Set
SHELL when calling `$MAKE -e'.

diff --git a/ChangeLog b/ChangeLog
index eccc3d1..e2577e7 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2002-04-10  Alexandre Duret-Lutz  <duret_g@epita.fr>
 
+       * Makefile.am (maintainer-check): Make sure `$MAKE -e' is
+       always followed by a SHELL setting.
+       * tests/ansi3.test, tests/ansi5.test, tests/subobj3.test: Set
+       SHELL when calling `$MAKE -e'.
+
+2002-04-10  Alexandre Duret-Lutz  <duret_g@epita.fr>
+
        * tests/subobj9.test: Use configure.in, not configure.ac,
        otherwise this fails with old libtool versions.
 

diff --git a/Makefile.am b/Makefile.am
index 17765b3..5e214b4 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -179,10 +179,17 @@ maintainer-check: automake aclocal
          exit 1; \
        fi
        @if grep -v SHELL $(srcdir)/tests/*.test | grep '\$$MAKE .*=' ; then \
-         echo 'Rewrite "$$MAKE foo=bar" as "foo=bar $$MAKE -e" in the above lines,' 1>&2; \
+         echo 'Rewrite "$$MAKE foo=bar" as "foo=bar $$MAKE -e SHELL=/bin/sh" in the above lines,' 1>&2; \
          echo 'it is more portable.' 1>&2; \
          exit 1; \
        fi
+## On NetBSD (1.5) and OSF, the $SHELL variable is still inherited from
+## the environment.
+       @if grep '$$MAKE.*-e' $(srcdir)/tests/*.test | grep -v '-e.*SHELL' ; then \
+         echo 'Always overwrite SHELL when using "$$MAKE -e".' 1>&2; \
+         echo 'Use "$$MAKE -e SHELL=/bin/sh" in the above lines.' 1>&2; \
+         exit 1; \
+       fi
        @if egrep 'SHELL=.*\$$MAKE' $(srcdir)/tests/*.test; then \
          echo '$$MAKE ignores the SHELL envvar, use "$$MAKE SHELL=/bin/sh" in' 1>&2; \
          echo 'the above lines.' 1>&2; \

diff --git a/Makefile.in b/Makefile.in
index 94c4aae..0f8fadb 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -682,10 +682,15 @@ maintainer-check: automake aclocal
          exit 1; \
        fi
        @if grep -v SHELL $(srcdir)/tests/*.test | grep '\$$MAKE .*=' ; then \
-         echo 'Rewrite "$$MAKE foo=bar" as "foo=bar $$MAKE -e" in the above lines,' 1>&2; \
+         echo 'Rewrite "$$MAKE foo=bar" as "foo=bar $$MAKE -e SHELL=/bin/sh" in the above lines,' 1>&2; \
          echo 'it is more portable.' 1>&2; \
          exit 1; \
        fi
+       @if grep '$$MAKE.*-e' $(srcdir)/tests/*.test | grep -v '-e.*SHELL' ; then \
+         echo 'Always overwrite SHELL when using "$$MAKE -e".' 1>&2; \
+         echo 'Use "$$MAKE -e SHELL=/bin/sh" in the above lines.' 1>&2; \
+         exit 1; \
+       fi
        @if egrep 'SHELL=.*\$$MAKE' $(srcdir)/tests/*.test; then \
          echo '$$MAKE ignores the SHELL envvar, use "$$MAKE SHELL=/bin/sh" in' 1>&2; \
          echo 'the above lines.' 1>&2; \

diff --git a/tests/ansi3.test b/tests/ansi3.test
index ef7e7fb..0f62ff9 100755 (executable)
--- a/tests/ansi3.test
+++ b/tests/ansi3.test
@@ -41,5 +41,5 @@ $ACLOCAL \
    && $AUTOCONF \
    && $AUTOMAKE -a \
    && CC='gcc' ./configure \
-   && ANSI2KNR=./ansi2knr U=_ $MAKE -e \
+   && ANSI2KNR=./ansi2knr U=_ $MAKE -e SHELL=/bin/sh \
    && ./hello

diff --git a/tests/ansi5.test b/tests/ansi5.test
index f9d56b5..9659b4a 100755 (executable)
--- a/tests/ansi5.test
+++ b/tests/ansi5.test
@@ -73,6 +73,6 @@ done
 mv one/Makefile one/Makefile.old
 sed -e 's,ANSI2KNR =,ANSI2KNR = ./ansi2knr,' < one/Makefile.old > one/Makefile
 
-U=_ $MAKE -e
+U=_ $MAKE -e SHELL=/bin/sh
 ./one/joe
 ./two/maude

diff --git a/tests/subobj3.test b/tests/subobj3.test
index 9be2fa8..e39ff55 100755 (executable)
--- a/tests/subobj3.test
+++ b/tests/subobj3.test
@@ -45,5 +45,5 @@ $ACLOCAL \
    && $AUTOCONF \
    && $AUTOMAKE -a \
    && CC='gcc' ./configure \
-   && ANSI2KNR=./ansi2knr U=_ $MAKE -e \
+   && ANSI2KNR=./ansi2knr U=_ $MAKE -e SHELL=/bin/sh \
    && ./hello