Closed
Bug 475706
Opened 16 years ago
Closed 16 years ago
Release l10n repacks builds en-US
Categories
(Release Engineering :: General, defect, P2)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: coop)
References
Details
Attachments
(1 file)
|
508 bytes,
patch
|
armenzg
:
review+
nthomas
:
checked-in+
|
Details | Diff | Splinter Review |
eg http://staging-master.build.mozilla.org:8010/builders/linux_repack/builds/1300
Probably because release use shipped-locales which does include en-US, while the nightly builds use all-locales which doesn't.
Options
1) Fix L10nMixin::GetLocales to exclude en-US
2) Remove en-US from shipped-locales
1 is obviously the right thing to do, 2) might cause bustage elsewhere.
|
Assignee | |
Updated•16 years ago
|
Assignee: nobody → ccooper
Status: NEW → ASSIGNED
Priority: -- → P2
|
|
Assignee | |
Comment 1•16 years ago
|
||
Rather than mess with the lambdas, I just skip past en-US in the callback when creating the buildsets.
Attachment #359300 -
Flags: review?(armenzg)
|
Reporter | |
Comment 2•16 years ago
|
||
Looks good to me. Would like to land this asap.
|
||
Updated•16 years ago
|
Attachment #359300 -
Flags: review?(armenzg) → review+
|
|
Reporter | |
Comment 3•16 years ago
|
||
Comment on attachment 359300 [details] [diff] [review]
Skip en-US
Went ahead and checked this in for another test run.
Attachment #359300 -
Flags: checked‑in+ checked‑in+
|
|
Reporter | |
Comment 4•16 years ago
|
||
Thanks for fixing this coop.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
|
||
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•