[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-21 Thread Paul D. Smith
Follow-up Comment #8, bug #63821 (project make): Yes I'm going to go through and remove them all as a cleanup, not until after this release though. ___ Reply to this item at: _

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-21 Thread Dmitry Goncharov
Follow-up Comment #7, bug #63821 (project make): > We can reconsider it. There is no need. It is all good. I see that you removed parameter void from install_default_suffix_rules definiton. Going forward, do you want empty paranthesis rather than void in functions definition? ___

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Paul D. Smith
Update of bug #63821 (project make): Status:None => Fixed Open/Closed:Open => Closed Component Version: SCM => 4.3 Fixed Release:

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Dmitry Goncharov
Follow-up Comment #5, bug #63821 (project make): > The title is slightly misleading Hopefully, subsequent description made it clear. > Curious why you decided to install the default suffix rules after snap_deps() @dgoncharov . i wanted to install default suffix rules as close to convert_to_patt

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Paul D. Smith
Follow-up Comment #4, bug #63821 (project make): Curious why you decided to install the default suffix rules after snap_deps() @dgoncharov . That doesn't feel right to me, was there something that didn't work right if you did it between defining the makeflags and snap_deps(), rather than after?

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Paul D. Smith
Follow-up Comment #3, bug #63821 (project make): The title is slightly misleading because MAKEFLAGS += -r DOES disable default suffix rules, it just doesn't remove them if you add back the extensions to the .SUFFIXES target. See this test from tests/scripts/options/dash-r which passes: # Make s

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Dmitry Goncharov
Follow-up Comment #2, bug #63821 (project make): A correction: After parsing makefiles the patch appends the default suffix rules to suffix_file->deps only if there is no user defined rules in the makefile. ___ Reply to this item at:

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Dmitry Goncharov
Additional Item Attachment, bug #63821 (project make): File name: sv63821_fix.diff Size:3 KB File name: sv63821_test.diff Size:2 KB

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Dmitry Goncharov
Follow-up Comment #1, bug #63821 (project make): Setting -r in MAKEFLAGS in makefile fails to the disable default suffix rules. $ ls hello.c makefile $ cat makefile MAKEFLAGS+=-r .SUFFIXES: .c .o all: hello.o $ make-4.4 cc-c -o hello.o hello.c $ $ rm hello.o $ make-4.4 -r make-4.4: ***

[bug #63821] Switch -r fails to disable default suffix rules.

2023-02-20 Thread Dmitry Goncharov
URL: Summary: Switch -r fails to disable default suffix rules. Group: make Submitter: dgoncharov Submitted: Mon 20 Feb 2023 05:52:08 PM UTC Severity: 3 - Normal