automake
[Top][All Lists]
Advanced

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

Re: 67-bigger-handle-factored-dependencies.patch


From: Tom Tromey
Subject: Re: 67-bigger-handle-factored-dependencies.patch
Date: 24 Feb 2001 21:22:10 -0700

>>>>> "Akim" == Akim Demaille <address@hidden> writes:

Akim>   * automake.in (&handle_merge_targets, &do_one_merge_target): Don't
Akim>   work on targets handled by %dependencies.
Akim>   (&handle_factored_dependencies): Do it.
Akim>   * install.am: Install install-exec, install-data and uninstall's
Akim>   suites.

This is ok.

Akim> +    # Install the -local hooks.
Akim> +    foreach (keys %dependencies)
Akim> +    {
Akim> +      # Hooks are installed on the -am targets.
Akim> +      s/-am$// or next;

What is the `dist-all' target used for?  I see it is collected in
%dependencies.

Does it really make sense for us to also have `clean' hooks?  This
seems pointless.  I think your patch adds this.  If you do that please
update the documentation.

Akim> +    # FIXME: Why not be as liberal as we are with -local hooks?

Sometimes there's no point.

Akim> +    foreach ('install-exec', 'install-data')

dist-hook could also be handled here.

Tom



reply via email to

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