Bug 1449624 - Make single-locale repacks use correct locale. r=mshal draft
authorNick Alexander <nalexander@mozilla.com>
Tue, 20 Mar 2018 17:00:38 -0700
changeset 780620 db48d0e411d2d73b3210c7c83441a4cf58b822ca
parent 779856 0a2dae2d8cf9f628c55668514c54a23da446d5de
push id106042
push usernalexander@mozilla.com
push dateWed, 11 Apr 2018 16:35:47 +0000
reviewersmshal
bugs1449624, 1443208
milestone61.0a1
Bug 1449624 - Make single-locale repacks use correct locale. r=mshal This was an oversight in the landing of Bug 1443208. We need to set AB_CD explicitly for the invoked target, and, due to the way that l10n.mk is written, we also need to set the additional internal variables IS_LANGUAGE_REPACK and REAL_LOCALE_MERGEDIR. MozReview-Commit-ID: 29lQXYiOf6a
mobile/android/locales/Makefile.in
--- a/mobile/android/locales/Makefile.in
+++ b/mobile/android/locales/Makefile.in
@@ -67,17 +67,17 @@ langpack: langpack-$(AB_CD)
 
 # This is a generic target that will make a langpack and repack tarball
 # builds. It is called from the tinderbox scripts. Alter it with caution.
 
 installers-%: IS_LANGUAGE_REPACK=1
 installers-%:
 	$(MAKE) clobber-stage
 	$(MAKE) libs-$*
-	$(MAKE) -C $(DEPTH)/mobile/android/base android_apks
+	$(MAKE) -C $(DEPTH)/mobile/android/base android_apks AB_CD=$* IS_LANGUAGE_REPACK=1 REAL_LOCALE_MERGEDIR=$(CURDIR)/merge-dir/$*
 	$(MAKE) package-langpack-$*
 	$(MAKE) repackage-zip-$*
 	@echo 'repackaging done'
 
 # When we unpack fennec on MacOS X the platform.ini and application.ini are in slightly
 # different locations that on all other platforms
 ifeq (Darwin, $(OS_ARCH))
 GECKO_PLATFORM_INI_PATH='$(STAGEDIST)/platform.ini'