automake-patches
[Top][All Lists]
Advanced

[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



reply via email to

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