automake-patches
[Top][All Lists]
Advanced

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

Re: proposed patch: parallel-tests: per-extension test driver: ext_COMPI


From: Ralf Wildenhues
Subject: Re: proposed patch: parallel-tests: per-extension test driver: ext_COMPILE.
Date: Sat, 28 Mar 2009 20:44:49 +0100
User-agent: Mutt/1.5.18 (2008-05-17)

* Ralf Wildenhues wrote on Tue, Mar 24, 2009 at 10:57:16PM CET:
> How about this?  I really appreciate a look over this.
> 
> Note that for extension .foo, the variable names will be FOO_LOG_*,
> i.e., remove the dot, then uppercase the rest.
> 
> I chose to go all the way and add AM_*FLAGS too.
> 
> One could, as a generalization, even add, for the non-extension rules,
> add
>   LOG_COMPILER = $(LOG_COMPILE) $(AM_LOG_FLAGS) $(LOG_FLAGS)
> 
> but I haven't done that yet.  WDYT?

For now, I have pushed the patch as-is to the ad-parallel-tests branch.

Cheers,
Ralf

>     parallel-tests: per-extension test driver: <EXT>_LOG_COMPILER.
>     
>     For test files with extension <ext>, introduce the internal
>     variable <EXT>_LOG_COMPILE, which expands to
>     $(<EXT>_LOG_COMPILER) $(AM_<EXT>_LOG_FLAGS) $(<EXT>_LOG_FLAGS).
>     Turn also the lib/Automake/tests testsuite over to the new
>     test driver.
>     * doc/automake.texi (Tests): Document `EXT_LOG_COMPILER' and
>     `EXT_LOG_FLAGS'.
>     * lib/am/check2.am: Insert `%COMPILE%' right before test.
>     * automake.in (handle_tests): Substitute `COMPILE' for check2,
>     empty for tests without extension, and `$(ext_LOG_COMPILE)' for
>     extension `ext'.  In the latter case, define it from the public
>     components.
>     * configure.ac (AM_INIT_AUTOMAKE): Use `parallel-test' globally.
>     * tests/Makefile.am (AUTOMAKE_OPTIONS): Remove, not needed here
>     any more.
>     * lib/Automake/tests/Makefile.am (TESTS_ENVIRONMENT): Split ...
>     (PL_LOG_COMPILER, PL_LOG_FLAGS): ... into these new variables.
>     (TESTS_EXTENSIONS): New variable, initialize to `.pl'.
>     * tests/parallel-tests7.test: New test.
>     * tests/Makefile.am: Update.
>     Suggestion by Akim Demaille.




reply via email to

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