diff options
author | Dmitry Goncharov <dgoncharov@users.sf.net> | 2023-02-20 15:17:24 -0500 |
---|---|---|
committer | Paul Smith <psmith@gnu.org> | 2023-02-20 15:17:24 -0500 |
commit | e9dd614d73fc625845a0de089edae3a8433e30c1 (patch) | |
tree | f87efeed8549e634939f513131b0f945619e53f8 /src | |
parent | f5dc17ac2dcf9093d15c67facc8a481212c4c360 (diff) | |
download | make-git-e9dd614d73fc625845a0de089edae3a8433e30c1.tar.gz |
[SV 63821] Don't set up default suffix rules if makefile sets -r
When built-in rules are disabled by adding -r to MAKEFLAGS in the
makefile, don't add suffix rules at all so that if suffixes are
added back via .SUFFIXES, the rules are still not there.
* src/main.c (main): Set default suffix rules after parsing makefiles.
* src/default.c (install_default_suffix_rules): Install a default
suffix rule only if there is no user defined rule.
* tests/scripts/features/suffixrules: Add tests.
Diffstat (limited to 'src')
-rw-r--r-- | src/default.c | 20 | ||||
-rw-r--r-- | src/main.c | 12 |
2 files changed, 16 insertions, 16 deletions
diff --git a/src/default.c b/src/default.c index ffb5f98c..e396269b 100644 --- a/src/default.c +++ b/src/default.c @@ -707,7 +707,7 @@ set_default_suffixes (void) installed after. */ void -install_default_suffix_rules (void) +install_default_suffix_rules () { const char **s; @@ -717,14 +717,16 @@ install_default_suffix_rules (void) for (s = default_suffix_rules; *s != 0; s += 2) { struct file *f = enter_file (strcache_add (s[0])); - /* This function should run before any makefile is parsed. */ - assert (f->cmds == 0); - f->cmds = xmalloc (sizeof (struct commands)); - f->cmds->fileinfo.filenm = 0; - f->cmds->commands = xstrdup (s[1]); - f->cmds->command_lines = 0; - f->cmds->recipe_prefix = RECIPEPREFIX_DEFAULT; - f->builtin = 1; + /* Install the default rule only if there is no user defined rule. */ + if (!f->cmds) + { + f->cmds = xmalloc (sizeof (struct commands)); + f->cmds->fileinfo.filenm = NULL; + f->cmds->commands = xstrdup (s[1]); + f->cmds->command_lines = NULL; + f->cmds->recipe_prefix = RECIPEPREFIX_DEFAULT; + f->builtin = 1; + } } } @@ -2028,13 +2028,6 @@ main (int argc, char **argv, char **envp) /* Define the initial list of suffixes for old-style rules. */ set_default_suffixes (); - /* Define the file rules for the built-in suffix rules. These will later - be converted into pattern rules. We used to do this in - install_default_implicit_rules, but since that happens after reading - makefiles, it results in the built-in pattern rules taking precedence - over makefile-specified suffix rules, which is wrong. */ - install_default_suffix_rules (); - /* Define some internal and special variables. */ define_automatic_variables (); @@ -2280,6 +2273,11 @@ main (int argc, char **argv, char **envp) define_makeflags (0); + /* Define the file rules for the built-in suffix rules. These will later + be converted into pattern rules. */ + + install_default_suffix_rules (); + /* Make each 'struct goaldep' point at the 'struct file' for the file depended on. Also do magic for special targets. */ |