[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Infinite recursion with `check_SCRIPTS = check'
From: |
Ralf Wildenhues |
Subject: |
Re: Infinite recursion with `check_SCRIPTS = check' |
Date: |
Sun, 21 Oct 2007 11:46:28 +0200 |
User-agent: |
Mutt/1.5.16 (2007-10-11) |
Hello Benoit,
* Benoit SIGOURE wrote on Sat, Oct 20, 2007 at 01:58:23PM CEST:
> Hello list,
> Due to a sudden lack of imagination, I wrote a testsuite named `check' and
> used it as a `check_SCRIPTS' which led to an infinite recursion of make
> (because the .PHONY check depends on check-am which does a `$(MAKE)
> $(AM_MAKEFLAGS) $(check_SCRIPTS)' so the make has no chance of catching the
> dependency cycle).
>
> Maybe automake should warn (or bail out) in such a case. I can write a
> patch for that, if you think that could be something relevant.
Well, the user should be able to override the `check' target if he so
wants. If it's possible to reliably warn without giving false failures
without a lot of work, then it would help. But often such checks turn
out to be rather expensive in both developer time and automake run time.
> Maybe something along these lines would be better:
> diff --git a/automake.in b/automake.in
> index 3c47cc1..263674c 100755
> --- a/automake.in
> +++ b/automake.in
> @@ -6826,8 +6826,10 @@ sub am_primary_prefixes ($$@)
> # Handle `where_HOW' variable magic. Does all lookups, generates
> # install code, and possibly generates code to define the primary
> -# variable. The first argument is the name of the .am file to munge,
> -# the second argument is the primary variable (e.g. HEADERS), and all
> +# variable. The first argument can be one of: '-noextra', '-candist',
> +# or '-defaultdist'. FIXME: Document these options.
> +# The following argument is the name of the .am file to munge,
> +# the following argument is the primary variable (e.g. HEADERS), and all
> # subsequent arguments are possible installation locations.
Sorry, but I don't understand at all what you're trying to say here.
> If I have some time next week, I'll try to port the parallel testsuite
> thing that Akim wrote
That would be great!
Cheers,
Ralf