[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Separate failing part of test `all.test'.
From: |
Ralf Wildenhues |
Subject: |
Re: [PATCH] Separate failing part of test `all.test'. |
Date: |
Sun, 8 Aug 2010 12:36:18 +0200 |
User-agent: |
Mutt/1.5.20 (2010-04-22) |
* Stefano Lattarini wrote on Wed, Jul 14, 2010 at 01:36:57PM CEST:
> * tests/all.test: Run aclocal only once. Minor cosmetic changes.
> Move checks that several *-local's in a single rule work ...
This sentence does not make sense to me. Maybe a missing word?
> * tests/manylocal.test: ... in this new test.
> * tests/Makefile.am (TESTS): Updated.
> (XFAIL_TESTS): Remove `all.test', add `manylocal.test'.
Patch is OK, but I'd be happier if you could rename all.test to
all2.test and manylocal.test to all.test (with adjustments to
Makefile.am of course) so that
- it is clearer that all and all2 are related,
- comparison with old testsuite log results are still meaningful.
I have many old log files lying around on several systems, and I tend to
use them to find out quickly if some bug is long-standing on this system
or new.
Thanks,
Ralf
- Re: [PATCH] Separate failing part of test `all.test'.,
Ralf Wildenhues <=