D1167: makefile: add target to apply clang-format in-place

Matt Harbison mharbison72 at gmail.com
Thu Oct 26 21:14:27 EDT 2017


On Wed, 25 Oct 2017 23:42:04 -0400, av6 (Anton Shestakov)  
<phabricator at mercurial-scm.org> wrote:

> av6 added a comment.
>
>
>   I'm a bit late to the party, but wanted to remind people that targets  
> that are not actual files (in this case, a file named "format-c") should  
> also go into .PHONY list. Nothing bad will happen right now (unless such  
> file appears at some point), but adding it to "phony" targets helps  
> keeping makefile in proper shape. (Looking at the already unwieldy  
> .PHONY list I wish Make had another way to mark targets "phony").

Split up the .PHONY declaration and put it in front of each target, so  
it's more obvious when one is missing?

https://unix.stackexchange.com/questions/217295/phony-all-rules-in-gnu-make-file


More information about the Mercurial-devel mailing list