emacs-devel
[Top][All Lists]
Advanced

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

Re: reftex.el autoloads


From: Eli Zaretskii
Subject: Re: reftex.el autoloads
Date: Fri, 18 Dec 2015 23:27:42 +0200

> From: address@hidden (Phillip Lord)
> Date: Fri, 18 Dec 2015 20:55:24 +0000
> Cc: Glenn Morris <address@hidden>, Eli Zaretskii <address@hidden>,
>       address@hidden
> 
> > Here are a couple of other comments. That's not the right way to declare
> > autoloads-force to be phony.
> 
> I thought it was an usual usage, but I copied it from autoloads. I
> guess:
> 
> autoloads-force .PHONY:
>       rm loaddefs.el
>       $(MAKE) autoloads
> 
> should be
> 
> autoloads-force:
>         rm loaddefs.el
>         $(MAKE) autoloads
> 
> .PHONY: autoloads-force
> 
> 
> 
> > Also, there shouldn't need to be an autoloads-force at all: if some
> > other files need to be built they should be listed as dependencies of
> > autoload.
> 
> 
> Indeed. It's always been this way, though -- from a non-bootstrap on
> emacs-25 try

Any reason why you need explicit Makefile targets for the new files?
Why not model them on calc/calc-loaddefs.el, which has not target in
the Makefile at all?



reply via email to

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