Bug XYZ - Remove '10' argument to profileserver.py draft
authorMike Shal <mshal@mozilla.com>
Thu, 06 Dec 2018 16:38:20 -0500
changeset 1779799 51d623be3a08820319915ef1e6195ef849e64721
parent 1779798 9b43a77233bf8fecf29345bf6315c09f5c15a09c
child 1779800 072853493f603684e6e4b67d0b4173e392c7e537
push id320312
push usermshal@mozilla.com
push dateThu, 06 Dec 2018 21:47:37 +0000
treeherdertry@a693256f5242 [default view] [failures only]
bugs663251, 672563
milestone65.0a1
Bug XYZ - Remove '10' argument to profileserver.py Support for running profileserver.py multiple times was originally added in bug 663251 and subsequently removed in bug 672563. Since then we've been copying around the argument to profileserver.py from mozconfigs in buildbot-configs, to mozconfigs in mozilla-central, to client.mk, and ultimately to Makefile.in. It no longer does anything and can be removed. MozReview-Commit-ID: DsZkInTOyU
Makefile.in
--- a/Makefile.in
+++ b/Makefile.in
@@ -182,17 +182,17 @@ profiledbuild::
 	$(call BUILDSTATUS,TIER_START pgo_profile_generate)
 	$(MAKE) default MOZ_PROFILE_GENERATE=1
 	$(call BUILDSTATUS,TIER_FINISH pgo_profile_generate)
 	$(call BUILDSTATUS,TIER_START pgo_package)
 	$(MAKE) package MOZ_INTERNAL_SIGNING_FORMAT= MOZ_EXTERNAL_SIGNING_FORMAT=
 	rm -f jarlog/en-US.log
 	$(call BUILDSTATUS,TIER_FINISH pgo_package)
 	$(call BUILDSTATUS,TIER_START pgo_profile)
-	JARLOG_FILE=jarlog/en-US.log $(PYTHON) $(topsrcdir)/build/pgo/profileserver.py 10
+	JARLOG_FILE=jarlog/en-US.log $(PYTHON) $(topsrcdir)/build/pgo/profileserver.py
 	$(call BUILDSTATUS,TIER_FINISH pgo_profile)
 	$(call BUILDSTATUS,TIER_START pgo_clobber)
 	$(MAKE) maybe_clobber_profiledbuild
 	$(call BUILDSTATUS,TIER_FINISH pgo_clobber)
 	$(call BUILDSTATUS,TIER_START pgo_profile_use)
 	$(MAKE) default MOZ_PROFILE_USE=1 $(if $(CLANG_CL),MOZ_PROFILE_ORDER_FILE=$(topobjdir)/cygprofile.txt)
 	$(call BUILDSTATUS,TIER_FINISH pgo_profile_use)