Closed Bug 1383388 Opened 2 years ago Closed 2 years ago

Port bug 1362617 to C-C [Generalize MOZ_CHROME_MULTILOCALE to work for browser as well]


(Thunderbird :: Build Config, enhancement)

Not set


(Not tracked)

Thunderbird 58.0


(Reporter: Paenglab, Assigned: jorgk)




(3 files, 2 obsolete files)

No description provided.
Attached patch multilocale.patch (obsolete) — Splinter Review
Not tested but it should work. The path in


looks correct in my objdir.
Assignee: nobody → richard.marti
Attachment #8889038 - Flags: review?(jorgk)
Comment on attachment 8889038 [details] [diff] [review]

That got built.
Attachment #8889038 - Flags: review?(jorgk) → review+
Keywords: checkin-needed
Bug 1362617 was backed-out. See bug 1362617 comment 21.
Keywords: checkin-needed
Depends on: 1362617
The changes in bug 1362617 are now too complicated for my null makefile knowledge. This can someone do which knows what he does.
Assignee: richard.marti → nobody
Attachment #8889038 - Attachment is obsolete: true
Jorg - heads up. We're getting ready to land it.

The two differences I believe for this patch is that we now added `MOZ_CHROME_LOCALE_ENTRIES` to your locales/ which has the list of entries for each locale.

Based on the resources I see you removing from, I think you want to set it to:


and it'll populate in PKG_LOCALE_MANIFEST.

For `PKG_LOCALE_MANIFEST` instead of pointing it at toolkit, you'll now have it in the objdir matching your locales dir, so sth like:

DEFINES += -DPKG_LOCALE_MANIFEST=$(topobjdir)/{mail|suite|im}/locales/

Let me know if you need more help! I'll be happy to look at the patch
Thanks for the heads-up, Zibi. I really have no idea what I'm doing here, so perhaps Tom can take it from here.
Assignee: nobody → jorgk
Attachment #8921997 - Flags: review?(mozilla)
Attachment #8921997 - Flags: feedback?(gandalf)
Comment on attachment 8921997 [details] [diff] [review]
1383388-multilocale.patch (v2)

Attachment #8921997 - Flags: feedback?(gandalf) → feedback+
Comment on attachment 8921997 [details] [diff] [review]
1383388-multilocale.patch (v2)

Review of attachment 8921997 [details] [diff] [review]:

This look reasonable.
Attachment #8921997 - Flags: review?(mozilla) → review+
Pushed by
Port bug 1362617 to C-C [Generalize MOZ_CHROME_MULTILOCALE to work for browser as well]. r=zibi,tomprince CLOSED TREE
Closed: 2 years ago
Resolution: --- → FIXED
I had to land this right on top of a whole lot of bustage since bug 1362617 landed.
Target Milestone: --- → Thunderbird 58.0
That busted the build:

mozbuild.preprocessor.Error: ('/builds/slave/tb-c-cen-lx-000000000000000000/build/mail/installer/', 66, 'FILE_NOT_FOUND', '/builds/slave/tb-c-cen-lx-000000000000000000/build/objdir-tb/mail/locales/')
+DEFINES += -DPKG_LOCALE_MANIFEST=$(topobjdir)/mail/locales/

Who builds that?
Looks like M-C now settled for:
+DEFINES += -DPKG_LOCALE_MANIFEST=$(topobjdir)/browser/installer/

Zibi, any hints?
Flags: needinfo?(gandalf)
Try was successful, here the patch also for IB and SM.
Attachment #8923075 - Attachment is obsolete: true
Attachment #8923075 - Flags: feedback?(gandalf)
Flags: needinfo?(gandalf)
Pushed by
Follow-up: Adapt to landed M-C version. rs=bustage-fix CLOSED TREE
Oh, my fault. Yes, we had to change a couple things after I reviewed your patch and I failed to update it here. Glad it works now :)
The only doubt I had was whether
is enough or we should do it like FF

Works with the former.
The only big we moved is the entries in `/mail/installer/`, which for mail were:


are now generated from python *per locale*. For now it doesn't change much because we're building only single locale builds, but later, we may be able to just do `MOZ_CHROME_MULTILOCALE="fr pl de" ./mach package` and get chrome entries packaged for all of those locales.

So yes, in case of mail|im|suite you only need `MOZ_CHROME_LOCALE_ENTRIES=@RESPATH@/chrome/` :)
Attachment #8924679 - Flags: review?(mozilla) → review+
Pushed by
Followup to support building in m-c topdir as well; r=me
You need to log in before you can comment on or make changes to this bug.