automake-patches
[Top][All Lists]
Advanced

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

Re: lib/Automake/ test suite


From: Raja R Harinath
Subject: Re: lib/Automake/ test suite
Date: Mon, 18 Nov 2002 11:43:05 -0600
User-agent: Gnus/5.090008 (Oort Gnus v0.08) Emacs/21.3.50 (i686-pc-linux-gnu)

Hi,

Alexandre Duret-Lutz <address@hidden> writes:

>>>> "Raja" == Raja R Harinath <address@hidden> writes:
>
> [...]
>
>  Raja> Since this is purely a test of Automake::Conditional, should it be
>  Raja> moved into 'lib/Automake'?
>
>  Raja> Is it worth having an independent test directory for the Perl modules
>  Raja> alone?  Maybe even using Perl's own testing infrastructure.
>
> Here is my proposal for a lib/Automake/ test suite.  This also corrects
> a few things in ConditionalSet.pm.
>
> 2002-11-18  Alexandre Duret-Lutz  <address@hidden>
>
>       * lib/Automake/tests/Makefile.am, lib/Automake/tests/Conditional.pl,
>       lib/Automake/tests/ConditionalSet.pl: New files.
>       * lib/Automake/Makefile.am (SUBDIRS): New variable.
>       * configure.in: Output lib/Automake/tests/Makefile.
>       * lib/Automake/ConditionalSet.pm (by_conditions): Delete.
>       (conds): Don't use by_conditions, sort alphabetically, and
>       return sorted conds.
>       (string): Call ->string on each Conditional.
>       * tests/Makefile.am (TESTS): Remove cond12.test.
>       * tests/cond12.test: Remove, moved to
>       lib/Automake/tests/ConditionalSet.pl.
>       * tests/cond3.test: Adjust conditional ordering.

Looks OK.  Would be nice if the scripts understood the $VERBOSE
envvar, and became more talkative if it is set.

- Hari
-- 
Raja R Harinath ------------------------------ address@hidden




reply via email to

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