diff options
| author | Boris Kolpackov <boris@codesynthesis.com> | 2022-03-15 10:55:03 +0200 |
|---|---|---|
| committer | Boris Kolpackov <boris@codesynthesis.com> | 2022-03-15 10:55:03 +0200 |
| commit | 639c02bd1dc9892fecadb353b23d35028668183d (patch) | |
| tree | fd449abbe01af04cd64274ed4a64224fe8191bd1 /libbuild2/adhoc-rule-buildscript.cxx | |
| parent | 8c257da85cde2df8f459f0c7610445971fffb2a8 (diff) | |
Don't consider mtime of failed to unmatch prerequisite in ad hoc recipe
Diffstat (limited to 'libbuild2/adhoc-rule-buildscript.cxx')
| -rw-r--r-- | libbuild2/adhoc-rule-buildscript.cxx | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/libbuild2/adhoc-rule-buildscript.cxx b/libbuild2/adhoc-rule-buildscript.cxx index d91be65..77c0823 100644 --- a/libbuild2/adhoc-rule-buildscript.cxx +++ b/libbuild2/adhoc-rule-buildscript.cxx @@ -432,7 +432,8 @@ namespace build2 // If we managed to unmatch, blank it out so that it's not executed, // etc. Otherwise, convert it to ad hoc (we also automatically avoid - // hashing it and updating it during match in exec_depdb_dyndep()). + // hashing it, updating it during match in exec_depdb_dyndep(), and + // making us out of date in execute_update_prerequisites()). // // The hashing part is tricky: by not hashing it we won't detect the // case where it was removed as a prerequisite altogether. The @@ -1404,9 +1405,10 @@ namespace build2 target_state s (pt->executed_state (a)); rs |= s; - // Compare our timestamp to this prerequisite's. + // Compare our timestamp to this prerequisite's skipping + // update=unmatch. // - if (!e) + if (!e && (p.include & 4) == 0) { // If this is an mtime-based target, then compare timestamps. // |