Mark all targets in Doc/Makefile as PHONY

[sobolevn]

Some of them were missing fro PHONY list. Some of them were in correct order.
Now order of PHONY and order of definitions match.

Docs on PHONY: https://www.gnu.org/software/make/manual/html_node/Phony-Targets.html

[JulienPalard]

Thanks! We should keep a better eye on this one ☺

[zware]

I've found it easier to keep track of .PHONY targets by doing something like:

.PHONY: help
help:
	@echo ...

.PHONY: build
build:
	...

.PHONY: html
html: BUILDER = html
html: build
	@echo ...

It's more verbose, and I am not certain that all makes will handle it correctly, but I've found this style much easier to keep up to date. Perhaps a consideration for the future?

[JulienPalard]

Perhaps, yes.

Having the .PHONY near the rule is what I'm acustomed to do too, but I've seen many makefiles with all PHONY's on the same line like here.

I never heard about compatibility issues (but I'm only using GNU make so ... I'm not a reference).

[sobolevn]

I am also using .PHONY on target level, example: https://github.com/sobolevn/git-secret/blob/master/Makefile#L12

I never got any complaints :)

[sobolevn]

Thanks for the quick merge! 👍

[hugovk]

I've found it easier to keep track of .PHONY targets by doing something like:

.PHONY: help
help:
	@echo ...

.PHONY: build
build:
	...

.PHONY: html
html: BUILDER = html
html: build
	@echo ...

It's more verbose, and I am not certain that all makes will handle it correctly, but I've found this style much easier to keep up to date. Perhaps a consideration for the future?

Agreed, I expect it will happen again because it's normal to copy/paste a target and forget (or not know) to update the .PHONY line right at the top.

I was making some other fixes to the Makefile, so included this too: #98266