Bug 695812 - client.mk with RUN_CLIENT_PY doesn't work with mozconfigs that include from mozilla/. This should fix Mac builds on try server. r=Standard8,Callek
authorPhilippe M. Chiasson <gozer@mozilla.com>
Mon, 24 Oct 2011 12:05:45 +0100
changeset 9545 c005213082eb1d15ea95aa190975a40515f3a4b1
parent 9544 17379c903992b6d20fe0d0bb11c7060d0d83729d
child 9546 2ccfc6528ff2cd42dffab83f063357776e105ec7
push id281
push userbugzilla@standard8.plus.com
push dateWed, 21 Dec 2011 12:08:36 +0000
treeherdercomm-beta@6d973fa5c2e8 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersStandard8, Callek
bugs695812
Bug 695812 - client.mk with RUN_CLIENT_PY doesn't work with mozconfigs that include from mozilla/. This should fix Mac builds on try server. r=Standard8,Callek
client.mk
--- a/client.mk
+++ b/client.mk
@@ -170,23 +170,22 @@ CONFIGURES += $(TOPSRCDIR)/mozilla/js/sr
 
 #######################################################################
 # Rules
 
 # The default rule is build
 build::
 
 # These targets are candidates for auto-running client.py
-ifdef  ALWAYS_RUN_CLIENT_PY
-ifeq (0,${MAKELEVEL})
-build::                      run_client_py
-profiledbuild::              run_client_py
-configure::                  run_client_py
-endif
-endif
+
+ifeq (01,$(MAKELEVEL)$(if $(ALWAYS_RUN_CLIENT_PY),1,))
+
+build profiledbuild configure:: run_client_py
+	$(MAKE) -f $(TOPSRCDIR)/client.mk $@
+else
 
 
 # Print out any options loaded from mozconfig.
 all build clean depend distclean export libs install realclean::
 	@if test -f .mozconfig.out; then \
 	  cat .mozconfig.out; \
 	  rm -f .mozconfig.out; \
 	else true; \
@@ -380,16 +379,17 @@ build alldep postflight::
 ifdef MOZ_POSTFLIGHT
 	set -e; \
 	for mkfile in $(MOZ_POSTFLIGHT); do \
 	  $(MAKE) -f $(TOPSRCDIR)/$$mkfile postflight TOPSRCDIR=$(TOPSRCDIR) OBJDIR=$(OBJDIR) MOZ_OBJDIR=$(MOZ_OBJDIR); \
 	done
 endif
 
 endif # MOZ_CURRENT_PROJECT
+endif # RAN_CLIENT_PY
 
 ####################################
 # Postflight, after building all projects
 
 build alldep postflight_all::
 ifeq (,$(MOZ_CURRENT_PROJECT)$(if $(MOZ_POSTFLIGHT_ALL),,1))
 # Don't run postflight_all for individual projects in multi-project builds
 # (when MOZ_CURRENT_PROJECT is set.)