This is the mail archive of the
automake@gnu.org
mailing list for the automake project.
Re: 11-more-phony.patch
- To: Akim Demaille <akim at epita dot fr>
- Subject: Re: 11-more-phony.patch
- From: Tom Tromey <tromey at redhat dot com>
- Date: 28 Jan 2001 19:01:30 -0700
- Cc: Automake <automake at gnu dot org>
- List-Id: Discussion list for automake <automake.gnu.org><mailto:automake-request@gnu.org?subject=unsubscribe>
- References: <E14MuU0-0006Ug-00@nostromo.lrde.epita.fr>
- Reply-To: tromey at redhat dot com
>>>>> "Akim" == Akim Demaille <akim@epita.fr> writes:
Akim> * texinfos.am, tags.am, subdirs.am, multilib.am, mans.am: Add a
Akim> .PHONY target.
Akim> * automake.in (handle_texinfo, handle_man_pages, handle_multilib)
Akim> (handle_etags, handle_subdirs): Don't push into @phony.
This is ok. BTW I like what you are doing here and elsewhere --
cleaning up some of the engineering problems in automake.
Akim> + # FIXME: Suspicious name of phonies: there does not seem to be
Akim> + # any `uninstall-data-recursive' target for instance.
Akim> foreach $phonies ('', '-data', '-exec', 'dirs')
Akim> {
Akim> - push (@phony, 'install' . $phonies . '-recursive');
Akim> push (@phony, 'uninstall' . $phonies . '-recursive');
Akim> - }
I agree and I think it is safe to simply delete the entire loop. We
only have a single global `uninstall'; there are no -data or -exec
variants. Also there isn't an uninstalldirs target either. The only
remaining one is handled elsewhere in your patch.
Tom