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

[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_pattern as
possible. i think, it makes it easier to follow the logic in main.
install_default_suffix_rules, convert_to_pattern,
install_default_implicit_rules all belong together.

>  That doesn't feel right to me,

Are you thinking about the prior impl of snap_deps? We modified snap_deps and
removed that code that expanded prerequisites of .SUFFIXES. See
07eea3aa496184bb763b7306e9de6a40a94605c9.
Or do you see a use case that requires install_default_suffix_rules before
snap_deps?

> was there something that didn't work right if you did it between defining
the makeflags and snap_deps(), rather than after?

No. Either way it works the same, as far as i can see.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?63821>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/