Re: Automake 1.4j: Release Candidate [PATCH]

[Richard Boulton]

On Tue, Jul 31, 2001 at 11:06:01AM -0600, Tom Tromey wrote:
> Lars> Here's one.  The following Makefile.am doesn't generate any object
> Lars> dependencies.  $(libtest_a_OBJECTS) is always empty.
> 
> Thanks.  I turned this into a test case.  I'll fix it asap.

For what it's worth, and since I've looked at the dependency code fairly
recently, I took a look at this, and I think its the fault of the
@parent_conds argument in &variable_conditions_sub().  This actually looks
like outdated logic to me - duplicate or impossible conditionals are
removed later by &variable_conditions_reduce().

Removing the @parent_conds machinery and simply recursing through the
conditional variables and taking all the possible conditional combinations
makes sense to me, and works correctly as far as I can tell.  I attach a
patch to do this.  All tests behave as expected after this patch, except
for cond13.test which passes. :)

This may make things slightly less efficient, but the whole thing needs
replacing with something which doesn't do a combinatorial explosion as
the number of conditionals increases, anyway.  (As previously discussed on
the list.)

-- 
Richard

diffs
Description: Text document