Re: [PATCH] Testsuite: use $SHELL to run tests which are shell scripts.

[Ralf Wildenhues]

* Stefano Lattarini wrote on Mon, Jan 03, 2011 at 04:04:26PM CET:
> References:
>  <http://lists.gnu.org/archive/html/automake-patches/2010-12/msg00010.html>
>  <http://lists.gnu.org/archive/html/automake-patches/2010-09/msg00004.html>

> I have rebased the patch on latest master again.  I'll wait until *sunday*
> evening before pushing to master.
> 
> That said, even if the patch is pretty simple, it's not completely obvious,
> and a review before I push it would be very welcome.

I have some gripes with this patch.  I'm sorry, but can I ask you to
wait another couple of days?

Thanks,
Ralf

> Subject: [PATCH] Testsuite: use $SHELL to run tests which are shell scripts.
> 
> * tests/Makefile.am (TEST_LOG_COMPILER): Define so that the
> configure-time $SHELL is used to run the tests.
> (TEST_EXTENSIONS): Defined to `.test', for clarity.
> * tests/defs: Add code to re-execute by default the running
> test script with configure-time $SHELL.  Updated comments.
> * configure.ac: Check that `set -e' is working for $SHELL,
> not for /bin/sh.
> * tests/defs-static.in: Update comments.
> * tests/README (Supported shells): Updated.
> (Getting details from failures): Don't tell that tests are
> run by /bin/sh by default.