Re: [bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

On 5/19/24 18:25, Paul D. Smith wrote: Follow-up Comment #6, bug #60736 (group make): I think that using a warn option is better than forcing this to always warn. But I still think that the warning makes global usage useless, and since there's no way to control warnings on a per-target basis (to

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #6, bug #60736 (group make): I think that using a warn option is better than forcing this to always warn. But I still think that the warning makes global usage useless, and since there's no way to control warnings on a per-target basis (today) it means the warning is hard to use

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #5, bug #60736 (group make): Attached an alternative patch to https://savannah.gnu.org/bugs/index.php?65739. ___ Reply to this item at: __

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #4, bug #60736 (project make): The reason the exception exists is so that you can specify an extra target globally. You are testing a target-specific variable version, but if you set this globally: .EXTRA_PREREQS = foo bar foo: ; @echo $@ This will always generate a warning

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #3, bug #60736 (project make): > For one thing it causes a number of tests to fail. Fixed. > For another thing the docs explicitly say: > Note 'make' is smart enough not to add a prerequisite listed in '.EXTRA_PREREQS' as a prerequisite to itself. Does this behavior have t

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Additional Item Attachment, bug #60736 (project make): File name: sv_60736_add_avoid_circular_dep_msg_tests.diff Size:2 KB ___ Reply to

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #2, bug #60736 (project make): I think this change is not correct. For one thing it causes a number of tests to fail. For another thing the docs explicitly say: > Note 'make' is smart enough not to add a prerequisite listed in '.EXTRA_PREREQS' as a prerequisite to itself.

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Additional Item Attachment, bug #60736 (project make): File name: sv_60736_add_avoid_circular_dep_msg.diff Size:0 KB ___ Reply to this item at

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

Follow-up Comment #1, bug #60736 (project make): With the patch in the attachment the output is $ make -f makefile2 make: Circular hello.x <- hello.x dependency dropped. touch hello.x $ ___ Reply to this item at:

[bug #60736] Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps.

URL: Summary: Introduce "Circular <- dependency dropped." for .EXTRA_PREREQS deps. Project: make Submitted by: dgoncharov Submitted on: Sun 06 Jun 2021 01:33:46 AM UTC Severity: 3 -