libtool-patches
[Top][All Lists]
Advanced

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

Re: Tidy and fix clean and distclean rules for old testsuite.


From: Ralf Wildenhues
Subject: Re: Tidy and fix clean and distclean rules for old testsuite.
Date: Sun, 22 Aug 2010 12:29:16 +0200
User-agent: Mutt/1.5.20 (2010-04-22)

Hi Gary,

* Gary V. Vaughan wrote on Sun, Aug 22, 2010 at 06:46:55AM CEST:
> On 22 Aug 2010, at 11:23, Ralf Wildenhues wrote:
> > 
> > OK to apply?
> 
> Well, it is an improvement over what we have, so... sure, go ahead.

Thanks for the quick review!

> However, I continue to strongly dislike the old testsuite, and would
> much prefer to migrate it's tests into Autotest.

Oh.  I was going to propose a patch series to convert the old testsuite
to use the parallel-tests driver.  Despite the new one having grown
nicely, there are still several issues only found with the old one
(and some tests in the new one that probably need more work).

Oh well.  I think I'll still propose the patch series, since it
shouldn't make a future migration any worse.  I don't intend to work
on the migration myself; I prefer spending my time on other things.

> I understand the
> argument that our kludgy old tests are good for coverage of libtool
> with the bootstrapped autotools... but, honestly, I'd rather find
> out that the libtool I'm planning to install is going to fail with
> *my* autotools, than I would be told that the old testsuite still
> passes with whatever versions it happened to be bootstrapped with!

Sure.

> Getting rid of the old testsuite was actually my motivation for
> starting an Autotest based testsuite in the first place; and because
> bootstrap and reconf issues with the old testsuite have always been
> (and continue to be) a real pain.

And look what the new testsuite gained us: much better test coverage!
In that way I think it's a huge success even if it hasn't (yet) achieved
the goal you intended it for.  :-)

Cheers,
Ralf



reply via email to

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