Closed
Bug 723138
Opened 12 years ago
Closed 12 years ago
Language packs should work for all releases of a branch, maxVersion should use * for compatibility ranges
Categories
(MailNews Core :: Build Config, enhancement)
MailNews Core
Build Config
Tracking
(thunderbird12 fixed, thunderbird13 fixed, thunderbird-esr1012+ fixed)
RESOLVED
FIXED
Thunderbird 14.0
People
(Reporter: aryx, Assigned: standard8)
References
()
Details
(Whiteboard: [esr])
Attachments
(1 file, 1 obsolete file)
4.05 KB,
patch
|
Callek
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
standard8
:
approval-comm-esr10+
|
Details | Diff | Splinter Review |
The language packs provided in the release folder on the ftp should work for all releases of a branch and not only a single release. With Enterprise Support Releases (ESR), this should provide more benefits (but also for normal users). At the moment, the language pack have the same minVersion and maxVersion, but because the strings are frozen, compatibility ranges should look like 10.0 - 10.0.* (This is being done for Thunderbird 3.1.x)
Assignee | ||
Comment 1•12 years ago
|
||
Ok, I'm moving this to core build config, as I suspect it affects everyone, although I also suspect we'll need different sets of fixes across the board.
Product: Thunderbird → Core
QA Contact: build-config → build-config
Version: unspecified → Trunk
Assignee | ||
Updated•12 years ago
|
Whiteboard: [esr]
Assignee | ||
Comment 2•12 years ago
|
||
Ok, Commandeering this back to Mailnews for the c-c part of what I'm going to be posting in bug 723135.
Assignee: nobody → mbanner
Depends on: 723135
Product: Core → MailNews Core
QA Contact: build-config → build-config
Assignee | ||
Comment 3•12 years ago
|
||
This is the c-c version of the fix from bug 723135.
Attachment #601022 -
Flags: review?(bugspam.Callek)
Comment 4•12 years ago
|
||
Comment on attachment 601022 [details] [diff] [review] The fix I'm ok with this IFF the relevent m-c patch lands. One caveat to note though is that chemspill releases will get a weird maxversion in the langpacks of, e.g. 11.0.1.* rather than 11.0.* with your code here
Attachment #601022 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 601022 [details] [diff] [review] The fix Damn, I meant to account for that. Gonna have to rethink that a little.
Attachment #601022 -
Attachment is obsolete: true
Assignee | ||
Comment 6•12 years ago
|
||
Ok, this matches the patch in bug 723135. I'm hoping to land this on Monday, so that we've got it in time for the various releases.
Attachment #614777 -
Flags: review?(bugspam.Callek)
Updated•12 years ago
|
Attachment #614777 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 8•12 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/5c06bd39f172
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•12 years ago
|
||
This also required a bustage fix as bug 723135 hasn't landed yet: http://hg.mozilla.org/comm-central/rev/74679c8cb51d
Assignee | ||
Comment 10•12 years ago
|
||
Comment on attachment 614777 [details] [diff] [review] The fix v2 [Triage Comment] Moving forward across releases, as I want the test pilot fix for TB 12, and the langpack fixes for ESR.
Attachment #614777 -
Flags: approval-comm-esr10+
Attachment #614777 -
Flags: approval-comm-beta+
Attachment #614777 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 11•12 years ago
|
||
Checked in: http://hg.mozilla.org/releases/comm-aurora/rev/d59ce82bff2d http://hg.mozilla.org/releases/comm-aurora/rev/985be5ac4630
status-thunderbird13:
--- → fixed
Assignee | ||
Comment 12•12 years ago
|
||
Checked in: http://hg.mozilla.org/releases/comm-beta/rev/b1002bdd09b6 http://hg.mozilla.org/releases/comm-beta/rev/b0b116bdec07 http://hg.mozilla.org/releases/comm-esr10/rev/3b63dc8cfe1a http://hg.mozilla.org/releases/comm-esr10/rev/4bb3fe25be5f
status-thunderbird12:
--- → fixed
status-thunderbird-esr10:
--- → fixed
Comment 13•12 years ago
|
||
To be clear, I'm a bit surprised you self-approved and used your approval for the suite/* changes here. That said, I would have approved for suite had it been asked, so no end-result harm done.
Assignee | ||
Comment 14•12 years ago
|
||
Sorry, I'd forgotten about the suite parts.
Updated•12 years ago
|
Target Milestone: --- → Thunderbird 14.0
Assignee | ||
Updated•12 years ago
|
tracking-thunderbird-esr10:
--- → 12+
You need to log in
before you can comment on or make changes to this bug.
Description
•