enable os x l10n builds on oak

RESOLVED FIXED

Status

Release Engineering
General Automation
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bhearsum, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
We need to make sure all of the packaging changes that are being made for v2 signatures work with l10n repacks. Spohl has been pushing his patches for this to Oak, so we should enable them there.
(Reporter)

Comment 1

3 years ago
Created attachment 8473009 [details] [diff] [review]
enable l10n for mac only
Attachment #8473009 - Flags: review?(mshal)
Comment on attachment 8473009 [details] [diff] [review]
enable l10n for mac only

>diff --git a/mozilla/project_branches.py b/mozilla/project_branches.py
>--- a/mozilla/project_branches.py
>+++ b/mozilla/project_branches.py
>@@ -229,6 +229,13 @@ PROJECT_BRANCHES = {
>                 'nightly_signing_servers': 'nightly-signing',
>             },
>         },
>+        # OS X l10n repacks temporarily enabled for bug 1053797
>+        'enable_l10n': True,
>+        'enable_l10n_onchange': False,
>+        'l10n_platforms': ['macosx64'],

So this is specific to macosx64? Or did you want 'macosx' as well?

>+        'l10n_tree': 'fxcentral',
>+        'l10n_repo_path': 'l10n-central',
>+        'enUS_binaryURL': '/nightly/latest-oak',

It looks like the values for enUS_binaryURL in mozilla/config.py all have GLOBAL_VARS['download_base_url'] as a prefix. Do we need that here too or does that get added automatically elsewhere?
Attachment #8473009 - Flags: review?(mshal) → review+
(Reporter)

Comment 3

3 years ago
(In reply to Michael Shal [:mshal] from comment #2)
> Comment on attachment 8473009 [details] [diff] [review]
> enable l10n for mac only
> 
> >diff --git a/mozilla/project_branches.py b/mozilla/project_branches.py
> >--- a/mozilla/project_branches.py
> >+++ b/mozilla/project_branches.py
> >@@ -229,6 +229,13 @@ PROJECT_BRANCHES = {
> >                 'nightly_signing_servers': 'nightly-signing',
> >             },
> >         },
> >+        # OS X l10n repacks temporarily enabled for bug 1053797
> >+        'enable_l10n': True,
> >+        'enable_l10n_onchange': False,
> >+        'l10n_platforms': ['macosx64'],
> 
> So this is specific to macosx64? Or did you want 'macosx' as well?

"macosx64" is the platform name for our opt builds. It's confusing, I know - this is a historical artifact.

> >+        'l10n_tree': 'fxcentral',
> >+        'l10n_repo_path': 'l10n-central',
> >+        'enUS_binaryURL': '/nightly/latest-oak',
> 
> It looks like the values for enUS_binaryURL in mozilla/config.py all have
> GLOBAL_VARS['download_base_url'] as a prefix. Do we need that here too or
> does that get added automatically elsewhere?

The latter, and also confusing: https://github.com/mozilla/build-buildbot-configs/blob/master/mozilla/config.py#L2547
(Reporter)

Updated

3 years ago
Attachment #8473009 - Flags: checked-in+
(Reporter)

Comment 4

3 years ago
This went to production yesterday.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.