Open
Bug 1465901
Opened 6 years ago
Updated 2 years ago
Expose set of locales to multi-locale builds as TC environment variables
Categories
(Firefox Build System :: General, enhancement)
Tracking
(Not tracked)
ASSIGNED
People
(Reporter: nalexander, Assigned: nalexander)
References
(Blocks 2 open bugs)
Details
Attachments
(1 obsolete file)
Right now, the set of locales that get packaged into a (mobile/android) multi-locale build is controlled by configuration files deep in the guts of mozharness https://searchfox.org/mozilla-central/source/testing/mozharness/configs/multi_locale and those all seem to redirect to another JSON file in the tree, e.g. https://searchfox.org/mozilla-central/source/testing/mozharness/configs/multi_locale/mozilla-central_android.json#5 points to https://searchfox.org/mozilla-central/source/mobile/locales/l10n-changesets.json We want to have all of the locale extraction, filtering, etc for multi-locale builds done in the taskgraph configuration, much like it's done for single-locale repacks now. See https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/transforms/l10n.py#323 This ticket tracks exposing the set of locales to the Task Cluster environment directly, so that multi-locale mozconfig files can include them directly. That would let us move toward the world sketched in https://bugzilla.mozilla.org/show_bug.cgi?id=1445701#c6
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
(In reply to Phabricator Automation from comment #1) > Created attachment 8982598 [details] > Bug 1465901 - Expose set of locales to multi-locale builds as TC environment > variables. Nick, this should expose it as a variable, I didn't do any mozharness changes with this, nor is the implementation as clean as it could be, but this would make it work. I'm open to better env variables or formats too, this is just what I came up with today.
Flags: needinfo?(nalexander)
Comment 3•6 years ago
|
||
triaging, assigning to callek since he attached patches
Assignee: nobody → bugspam.Callek
Assignee | ||
Comment 4•6 years ago
|
||
(In reply to Justin Wood (:Callek) from comment #2) > (In reply to Phabricator Automation from comment #1) > > Created attachment 8982598 [details] > > Bug 1465901 - Expose set of locales to multi-locale builds as TC environment > > variables. > > Nick, this should expose it as a variable, I didn't do any mozharness > changes with this, nor is the implementation as clean as it could be, but > this would make it work. I'm open to better env variables or formats too, > this is just what I came up with today. I've been out of this loop for a long time, so clearing the NI, since I'm not going to act on this any time soon. Sorry, Callek!
Flags: needinfo?(nalexander)
Updated•6 years ago
|
Version: Version 3 → 3 Branch
Assignee | ||
Comment 5•3 years ago
|
||
I was just about to file basically this ticket, so I'll repurpose it instead.
Blocks: 1760694
Assignee | ||
Updated•3 years ago
|
Assignee: bugspam.Callek → nalexander
Status: NEW → ASSIGNED
Updated•2 years ago
|
Attachment #8982598 -
Attachment is obsolete: true
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•