Re: multiple AM_CONDITIONALs doens't work in CVS Automake

[Lars J. Aas]

On Fri, Mar 09, 2001 at 01:53:17PM +0100, Lars J. Aas wrote:
: When I bootstrap with the current Automake CVS sources, the strudels (@s)
: between the conditionals are missing:
: 
: > @address@hidden@address@hidden = \

I found out that this bug goes all the way back to automake.in r1.915.
> revision 1.915
> date: 2001/02/25 18:42:41;  author: akim;  state: Exp;  lines: +73 -49
> Internally just store the conditionals as space separated lists of
> CONDITIONS (instead of @CONDITIONS@).
> * automake.in (&conditional_true_when, &variable_conditions_sub):
> Split conditions at spaces.
> (&handle_dependencies, &variable_conditions_permutations): Don't
> put @ around conditions.
> (&variable_conditions_cmp): There are no @ to strip.
> (&make_condition): New.
> (&define_pretty_variable, &read_main_am_file, &read_am_file): Use
> it.
> (&read_main_am_file, &read_am_file): Stop playing with @ by hand.
> Join @conditional_stack with spaces.
> (&read_main_am_file): Adjust the output of variables.
> Output `TRUE = true' under the condition `TEST' as address@hidden@TRUE =
> true' and no longer address@hidden@TRUE = @address@hidden'.
> (&variable_conditions_cmp): Rename as...
> (&by_condition): this.
> Sort in a human pleasant order.
> Use it everywhere a human can see conditions.
> (&variable_conditions_reduce): Don't sort conditions, that's
> pointless.
> * tests/cond.test, ctarget1.test, pluseq3.test: Strengthen.

The
> Join @conditional_stack with spaces.
doesn't seem to do it's job.

The diff is attached.

  Lars J
-- 
Innovation is one percent inspiration and ninetynine percent perspiration,
and in my case; twice that...  -- Norville Barnes, `The Hudsucker Proxy'

diff
Description: Text document