Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-18 Thread Dmitry Goncharov
On Wed, May 18, 2022 at 5:37 PM Jacob Kopczynski wrote: >> - it might make sense to state explicitly that order-only prereqs >> which are phony behave in all ways like standard prerequisites. This statement is simply incorrect. Order-only prerequisites which are phony do not behave like

Re: Make Docs Errata

2022-05-18 Thread Martin Dorey
> It says SECONDARY EXPANSION, while the first time the manual is referring to > the first one I think it means what it says. I'd try to suggest a rewording for clarity... but I'm not seeing how it could be clearer. Perhaps you could suggest one? From:

Make Docs Errata

2022-05-18 Thread dVaGaymer
Hi, It is probably not that important but It doesn't cost me anything to report it. In chapter -3.9 Secondary Expansion- at paragraph 7: It says SECONDARY EXPANSION, while the first time the manual is referring to the first one. (since the example shows how MAKE parses the first expansion and

Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-18 Thread Jacob Kopczynski
The thing that the docs refer to as "impose order" is not a single thing, but two. I would characterize a normal prerequisite as doing three things rather than two: - *update-marking: *cause a target to be marked out of date if the prereq is marked out of date - *require-existence: *require the

Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-18 Thread Paul Smith
On Wed, 2022-05-18 at 10:22 -0700, Jacob Kopczynski wrote: > I believe I understand. The name "order-only" is highly misleading > and should be changed - it does considerably more than "only" > "order"; the only thing it does not do is check the timestamp. As described in the docs there are only

Re: Potential Bug: `.PHONY` targets and order-only prerequisites

2022-05-18 Thread Jacob Kopczynski
I believe I understand. The name "order-only" is highly misleading and should be changed - it does considerably more than "only" "order"; the only thing it does not do is check the timestamp. This portion of the documentation *particularly* needs changing: > Occasionally, however, you have a