diff options
author | Melody Horn <melody@boringcactus.com> | 2021-04-13 20:53:44 -0600 |
---|---|---|
committer | Melody Horn <melody@boringcactus.com> | 2021-04-13 20:53:44 -0600 |
commit | 7857e9f871caaf0b95f9eba9c4cdefb5046a9b2a (patch) | |
tree | c668076c6af2ffa80b9022adf6f7369e5433ed60 | |
parent | 57db2fce6f3e476d34e65b7c311c12cc9f9b71fd (diff) | |
download | makers-7857e9f871caaf0b95f9eba9c4cdefb5046a9b2a.tar.gz makers-7857e9f871caaf0b95f9eba9c4cdefb5046a9b2a.zip |
spell SUFFIXES correctly smdh
-rw-r--r-- | src/makefile/input.rs | 32 |
1 files changed, 26 insertions, 6 deletions
diff --git a/src/makefile/input.rs b/src/makefile/input.rs index cb53966..6860e80 100644 --- a/src/makefile/input.rs +++ b/src/makefile/input.rs @@ -527,10 +527,11 @@ impl<'a, 'parent, R: BufRead> MakefileReader<'a, 'parent, R> { } } - fn special_target_has_prereq(&self, target: &str, name: &str) -> bool { + fn special_target_has_prereq(&self, target: &str, name: &str, empty_counts: bool) -> bool { match self.targets.get(target) { Some(target) => { - target.prerequisites.is_empty() || target.prerequisites.iter().any(|e| e == name) + (empty_counts && target.prerequisites.is_empty()) + || target.prerequisites.iter().any(|e| e == name) } None => false, } @@ -675,9 +676,9 @@ impl<'a, 'parent, R: BufRead> MakefileReader<'a, 'parent, R> { // don't interpret things like `.tmp: ; mkdir -p $@` as single-suffix rules let inference_match = inference_match.and_then(|inference| { - if self.special_target_has_prereq(".SUFFIXES", inference.s1) + if self.special_target_has_prereq(".SUFFIXES", inference.s1, false) && (inference.s2.is_empty() - || self.special_target_has_prereq(".SUFFIXES", inference.s2)) + || self.special_target_has_prereq(".SUFFIXES", inference.s2, false)) { Some(inference) } else { @@ -715,7 +716,7 @@ impl<'a, 'parent, R: BufRead> MakefileReader<'a, 'parent, R> { match self.targets.get_mut(target) { Some(old_target) if commands.is_empty() - && !(target == ".SUFIXES" && prerequisites.is_empty()) => + && !(target == ".SUFFIXES" && prerequisites.is_empty()) => { let new_prerequisites = prerequisites .iter() @@ -1074,14 +1075,33 @@ info: #[test] fn sdafjijsafjdoisdf() -> R { - let file = "\ + let file = " cursed: \techo this uses the bash variable '$$#' and all that \\ \techo yeah its value is $$# and it's really cool "; let args = Args::empty(); let makefile = MakefileReader::read(&args, MacroSet::new(), Cursor::new(file), "")?; + let _makefile = makefile.finish(); + Ok(()) + } + + #[test] + fn double_suffix_rule() -> R { + let file = " +.c.o: +\techo yeet +.SUFFIXES: +.l.a: +\techo hey +.SUFFIXES: .test .post +.post.test: +\techo hiiii +"; + let args = Args::empty(); + let makefile = MakefileReader::read(&args, MacroSet::new(), Cursor::new(file), "")?; let makefile = makefile.finish(); + assert_eq!(makefile.inference_rules.len(), 2); Ok(()) } } |