Follow up l10n repack changes for bug 1385227

RESOLVED FIXED

Status

RESOLVED FIXED
a year ago
8 months ago

People

(Reporter: Pike, Assigned: tomprince)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

a year ago
+++ This bug was initially created as a clone of Bug #1385227 +++

Bug 1385227 changed quite a bit how l10n repacks work on desktop, and should break tb l10n repacks.

The actual patch should be fairly trivial, I hope.

PS: the landing on m-c has missed today's nightly, so we'll see tomorrow if it sticks

Comment 1

a year ago
So who's going to be working on that?
Flags: needinfo?(mozilla)

Updated

a year ago
Flags: needinfo?(jorgk)

Updated

a year ago
Flags: needinfo?(jorgk)
(Assignee)

Comment 2

a year ago
Looks like this has been backed out. I look at it once it lands.
Assignee: nobody → mozilla
Flags: needinfo?(mozilla)
(Assignee)

Updated

10 months ago
Flags: needinfo?(mozilla)

Comment 3

10 months ago
In the c-c l10n win32 build after 2017-11-21, the installer does not start.
Comment hidden (mozreview-request)

Comment 5

10 months ago
mozreview-review
Comment on attachment 8935221 [details]
Bug 1392245: Make l10n repack builds do what they're supposed to do (port bug 1385227).

https://reviewboard.mozilla.org/r/206096/#review211768
Attachment #8935221 - Flags: review?(philipp) → review+

Comment 6

10 months ago
Pushed by mozilla@hocat.ca:
https://hg.mozilla.org/comm-central/rev/cea82e83ff44
Make l10n repack builds do what they're supposed to do (port bug 1385227); r=Fallen
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
(Assignee)

Updated

10 months ago
Flags: needinfo?(mozilla)
You need to log in before you can comment on or make changes to this bug.