From 76393466c525cd362c3b56613f67fdfb734822f2 Mon Sep 17 00:00:00 2001
From: Axel Hecht <axel@pike.org>
Date: Fri, 13 Oct 2017 16:46:40 +0200
Subject: [PATCH] bug 1408380, bustage fix for language pack defines, r=gandalf

Undoing some of my ideas to support l10n-merge for defines.inc.
We'll add that once compare-locales can actually merge them.

The culprit why this didn't work is that there's one level
of $(MAKE) invocation missing between the rule that
defines IS_LANGUAGE_REPACK and config.mk actually taking
that into account.

MozReview-Commit-ID: 2JJZrvAPhG0

--HG--
extra : rebase_source : 7984a98611b40d35c58b65194cedf1ac20127293
---
 toolkit/locales/l10n.mk | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/toolkit/locales/l10n.mk b/toolkit/locales/l10n.mk
index 612e44997c13..bcbfff573919 100644
--- a/toolkit/locales/l10n.mk
+++ b/toolkit/locales/l10n.mk
@@ -166,16 +166,14 @@ repackage-zip-%: unpack
 	@$(MAKE) repackage-zip AB_CD=$* ZIP_IN='$(ZIP_IN)'
 
 
-ifdef IS_LANGUAGE_REPACK
-MERGE_TK_FILE = $(firstword \
-  $(wildcard $(REAL_LOCALE_MERGEDIR)/$(subst /locales,,toolkit/locales)/$(1)) \
-  $(wildcard $(LOCALE_SRCDIR)/$(1)) \
-  $(call EXPAND_LOCALE_SRCDIR,toolkit/locales)/$(1) )
-else
-MERGE_TK_FILE = $(call EXPAND_LOCALE_SRCDIR,toolkit/locales)/$(1)
-endif
-
-LANGPACK_DEFINES = $(call MERGE_TK_FILE,defines.inc) $(call MERGE_DIR,defines.inc)
+LANGPACK_DEFINES = \
+  $(firstword \
+    $(wildcard $(call EXPAND_LOCALE_SRCDIR,toolkit/locales)/defines.inc) \
+    $(MOZILLA_DIR)/toolkit/locales/en-US/defines.inc) \
+  $(firstword \
+    $(wildcard $(LOCALE_SRCDIR)/defines.inc) \
+    $(srcdir)/en-US/defines.inc) \
+$(NULL)
 
 # Dealing with app sub dirs: If DIST_SUBDIRS is defined it contains a
 # listing of app sub-dirs we should include in langpack xpis. If not,