Hi Eric, * Eric Blake wrote on Mon, Jan 28, 2008 at 03:07:14PM CET: > Applied, after one more round of testing with m4 (there were some > additional impacts, like having to use $at_myself rather than $0 to > reinvoke the --list option of the testsuite to name failed tests in the > log). Also, in the process, I noticed that a standalone test (one where > atconfig is not present), the platform information was not captured in the > log; also the PATH sanitization was taking a long time on cygwin prior to > running the first test. Fixed as follows:
[...] > @@ -361,31 +361,6 @@ fi > ## ---------------------- ## > m4_divert_pop([PREPARE_TESTS])dnl back to DEFAULTS > > -# Load the config file. > -for at_file in atconfig atlocal > -do > - test -r $at_file || continue > - . ./$at_file || AS_ERROR([invalid content: $at_file]) > -done Isn't this an API change? It looks to me the at* files are now interpreted after the command line (untested). That seems wrong: the command line should override atlocal, if I do ./testsuite PERL=/some/other/perl Cheers, and thanks for working on this, Ralf
