diff options
| author | Gaëtan Gilbert | 2020-06-27 22:02:49 +0200 |
|---|---|---|
| committer | Gaëtan Gilbert | 2020-06-27 22:02:49 +0200 |
| commit | c9ca432c1457bec080f98180bc5ef12ebdf57863 (patch) | |
| tree | 1d1ed518e6b200d0b02b6cd44c39f482e6de6a65 /test-suite | |
| parent | 427fb9282dd76a6b48c37fc2cffcbc0af10ad37d (diff) | |
| parent | 9690b138b13c0bb591c8c4aa6bb18cff5c79f655 (diff) | |
Merge PR #12583: [test-suite] Fix dependencies of modules/ files
Reviewed-by: SkySkimmer
Diffstat (limited to 'test-suite')
| -rw-r--r-- | test-suite/Makefile | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/test-suite/Makefile b/test-suite/Makefile index d4ad438d61..59cc3e5a38 100644 --- a/test-suite/Makefile +++ b/test-suite/Makefile @@ -629,7 +629,14 @@ $(addsuffix .log,$(wildcard ideal-features/*.v)): %.v.log: %.v $(PREREQUISITELOG } > "$@" # Additional dependencies for module tests -$(addsuffix .log,$(wildcard modules/*.v)): %.v.log: modules/Nat.vo modules/plik.vo +COMMON_MODULE_DEPENDENCIES := modules/plik.v modules/Nat.v +# We exclude Nat.v.log and plik.v.log because these log files do not +# depend on having the corresponding .vo files built first, and we end +# up with pseudo-cyclic build rules if we don't exclude them (See +# COQBUG(https://github.com/coq/coq/issues/12582)). Additionally, we +# impose order-only dependencies to ensure that we won't rebuild the +# .vo files in the .log target after we've already built them. +$(addsuffix .log,$(filter-out $(COMMON_MODULE_DEPENDENCIES),$(wildcard modules/*.v))): %.v.log: $(COMMON_MODULE_DEPENDENCIES:.v=.vo) | $(COMMON_MODULE_DEPENDENCIES:.v=.v.log) modules/%.vo: modules/%.v $(HIDE)$(coqc) -R modules Mods $< |