Bug 1337986 - Build dump_syms before any syms target. draft
authorChris Manchester <cmanchester@mozilla.com>
Mon, 10 Apr 2017 10:27:49 -0700
changeset 559818 21967057ce6bb3f3489f6ab16a8e0978cb55ae03
parent 559817 54425cd67082201411a1b3cd988b9ca443515b2b
child 559819 396a5167bd4bfbf595d6ca6427a64fe94cfe6ad9
push id53227
push userbmo:cmanchester@mozilla.com
push dateMon, 10 Apr 2017 17:27:51 +0000
bugs1337986
milestone55.0a1
Bug 1337986 - Build dump_syms before any syms target. MozReview-Commit-ID: CXlmMVMhzmf
config/recurse.mk
--- a/config/recurse.mk
+++ b/config/recurse.mk
@@ -81,16 +81,21 @@ recurse_compile: $(syms_targets)
 endif
 endif
 
 # Create a separate rule that depends on every 'syms' target so that
 # symbols can be dumped on demand locally.
 .PHONY: recurse_syms
 recurse_syms: $(syms_targets)
 
+# Ensure dump_syms gets built before any syms targets, all of which depend on it.
+ifneq (,$(filter toolkit/crashreporter/google-breakpad/src/tools/%/dump_syms/host,$(compile_targets)))
+$(syms_targets): $(filter toolkit/crashreporter/google-breakpad/src/tools/%/dump_syms/host,$(compile_targets))
+endif
+
 # The compile tier has different rules from other tiers.
 ifneq ($(CURRENT_TIER),compile)
 
 # Recursion rule for all directories traversed for all subtiers in the
 # current tier.
 $(addsuffix /$(CURRENT_TIER),$(CURRENT_DIRS)): %/$(CURRENT_TIER):
 	$(call RECURSE,$(CURRENT_TIER),$*)