Closed Bug 1900059 Opened 6 months ago Closed 5 months ago

support github in multilocale android builds in gecko

Categories

(Release Engineering :: Release Automation: L10N, enhancement)

enhancement

Tracking

(firefox-esr115 fixed, firefox128 fixed)

RESOLVED FIXED
Tracking Status
firefox-esr115 --- fixed
firefox128 --- fixed

People

(Reporter: bhearsum, Assigned: bhearsum)

References

(Blocks 1 open bug)

Details

Attachments

(4 files)

Follow-up to https://bugzilla.mozilla.org/show_bug.cgi?id=1881272, as I just discovered that multilocale builds for Android in Gecko are busted with Github enabled, eg: https://treeherder.mozilla.org/jobs?repo=maple&selectedTaskRun=dv10-XZAQxuUu42EVUBAwA.0

This script ends up using the same mixin that was patched in https://bugzilla.mozilla.org/show_bug.cgi?id=1881272, so all we need to do is make gittool.py available to it.

The multil10n.py mozharness script that gets run as part of fx_desktop_build.py (which we use to make android builds...) ends up in a virtualenv without this installed. It's no longer needed anyways as Python 2 is long gone, so dropping the references to six is the best fix.

Attachment #9405201 - Attachment description: Bug 1900059: Git rid of references to 'six' in gittool.py to fix multil10n.py usage of gittool.py r?#releng-reviewers! → Bug 1900059: Get rid of references to 'six' in gittool.py to fix multil10n.py usage of gittool.py r?#releng-reviewers!
Blocks: 1900421
Pushed by bhearsum@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8754cb0fc46e Support l10n strings from github in multilocale build script r=releng-reviewers,jcristau https://hg.mozilla.org/integration/autoland/rev/f1f57cc26b0b Get rid of references to 'six' in gittool.py to fix multil10n.py usage of gittool.py r=releng-reviewers,jcristau,hneiva
Status: NEW → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Attachment #9407616 - Flags: approval-mozilla-esr115?
Attachment #9407617 - Flags: approval-mozilla-esr115?
Attachment #9407616 - Flags: approval-mozilla-esr115? → approval-mozilla-esr115+
Attachment #9407617 - Flags: approval-mozilla-esr115? → approval-mozilla-esr115+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: