autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: 96-testsuite-log.patch


From: Akim Demaille
Subject: Re: 96-testsuite-log.patch
Date: 15 Jan 2001 08:32:36 +0100
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Crater Lake)

>>>>> "Pavel" == Pavel Roskin <address@hidden> writes:

>> Simply run `testsuite' with the failed tests instead of running the
>> debug scripts, this avoids having zillions of banners `Testing
>> Blabla 2.13' (another means to avoid these banners is to have the
>> test suite succeed :-).

Pavel> Not quite clear what you mean. 
Pavel> I think, the testsuite should not be verbose unless "-v" was
Pavel> given, even if it fails. For the failed tests, the debug
Pavel> scripts with "-v" should be created, and the testsuite should
Pavel> be rerun for those tests with another option, which would write
Pavel> the verbose output to the log (or separate logs for every
Pavel> failed test).

Then you want to change the current behavior.  I'm not in favor of
this, it is good to display the verbose runs, it might help naive
users to understand some very broken things in their environment,
knowledgeable users to find something wrong with Autoconf without
having to run additional commands, and experts to save some typing.

What I meant was that since a few patches ago, when debug scripts were
rerun, you had something like

======================
Testing Autoconf 2.49a
======================
1. Testing some stuff...
  ....
1. OK.

=====================
All 1 tests succeeded
=====================
======================
Testing Autoconf 2.49a
======================
2. Testing some stuff...
  ....
2. OK.

=====================
All 1 tests succeeded
=====================
======================
Testing Autoconf 2.49a
======================
3. Testing some stuff...
  ....
3. OK.

=====================
All 1 tests succeeded
=====================


Now that we just rerun directly testsuite instead of each debug
script, we have

======================
Testing Autoconf 2.49a
======================
1. Testing some stuff...
  ....
1. OK.

2. Testing some stuff...
  ....
2. OK.

3. Testing some stuff...
  ....
3. OK.

=====================
All 3 tests succeeded
=====================


Pavel> I guess your patch is a step in that direction, but I'm not
Pavel> sure.

>> -# STDOUT and STDERR can be the special value `ignore', in which
>> case +# STDOUT and STDERR can be the special value `skip', in which
>> case

Pavel> This is wrong. 

Indeed, thanks.  query-replace is not smart enough ;)



reply via email to

[Prev in Thread] Current Thread [Next in Thread]