Re: 17-factored-dependencies.patch

[akim]

On Wed, Jan 31, 2001 at 11:22:15AM -0700, Tom Tromey wrote:
> >>>>> "Akim" == Akim Demaille <address@hidden> writes:
> 
> Akim>         * automake.in (%factored_dependencies): New.
> Akim>         (file_contents): Use it.
> Akim>         (handle_phony): Rename as...
> Akim>         (handle_factored_dependencies): this.
> Akim>         * subdirs.am: No need for convolved syntax to declare .PHONY.
> 
> Akim> +# Holds the dependencies of target which dependencies are factored.
> Akim> +# Typically, `.PHONY' will appear in plenty of *.am files, but must
> Akim> +# be output once.  Arguably all pure dependencies could be subject
> Akim> +# to this factorization, but it is not unpleasant to have paragraphs
> Akim> +# in Makefile: keeping related stuff altogether.
> Akim> +%dependencies =
> Akim> +(
> Akim> +    ".PHONY" => []
> Akim> +);
> 
> Shouldn't this processing be done on a per-Makefile basis?
> I think so.  That means this should be initialized in
> &initialize_per_input, not globally.

Definitely!  Thanks, there are important things I have not caught yet.