Closed
Bug 470392
Opened 16 years ago
Closed 15 years ago
Update Thunderbird installer for Unicode support and locale verification
Categories
(Thunderbird :: Installer, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
Attachments
(2 files, 4 obsolete files)
22.99 KB,
patch
|
philor
:
review+
|
Details | Diff | Splinter Review |
4.74 KB,
patch
|
philor
:
review+
|
Details | Diff | Splinter Review |
Spinoff of bug 305039 for Thunderbird.
Assignee | ||
Comment 1•16 years ago
|
||
Phil, is this going to require ifdefs? I also cleaned up some of the code and made it so the survey can't be launched when uninstalling for Vista and above... this is due to the uninstaller always running elevated and there being no way to de-elevate in Vista.
Attachment #353791 -
Flags: review?(philringnalda)
Assignee | ||
Comment 2•16 years ago
|
||
Attachment #353792 -
Flags: review?(philringnalda)
Assignee | ||
Comment 3•16 years ago
|
||
Comment on attachment 353791 [details] [diff] [review] patch for Unicode support rev1 btw: the Unicode support patch requires the patch from bug 305039
Assignee | ||
Updated•16 years ago
|
Status: NEW → ASSIGNED
Comment 4•16 years ago
|
||
So far, I've been wrong every single time I say anything about 1.9.1 ifdefs, but I wouldn't think so, unless you're predicting a long and painful baking period. If I'm answering the right question, whether we can live with it landing on mozilla-central, burning the Windows comm-central+mozilla-central box for a couple of days (oh, it's already on fire!), then landing on 1.9.1 together with this patch, at which point everything greens up, I'd sure think so. If I'm wrong, I can ifdef while I review tomorrow night: we've already gotten in your way too much lately :)
Assignee | ||
Updated•16 years ago
|
Attachment #353791 -
Flags: review?(philringnalda)
Assignee | ||
Comment 5•16 years ago
|
||
Comment on attachment 353791 [details] [diff] [review] patch for Unicode support rev1 Thanks Phil, I'm going to redistribute the two patches like I did for Seamonkey. Basically, some of the changes in the Unicode patch can be done in the locales patch and are just cleanup. The Unicode portion is the part that would cause the comm-central / mozilla-central builds to burn so we should be ok as long as we don't have to bake long which we shouldn't.
Assignee | ||
Updated•16 years ago
|
Attachment #353792 -
Flags: review?(philringnalda)
Assignee | ||
Comment 6•16 years ago
|
||
Holding off on review until I test this.
Attachment #353791 -
Attachment is obsolete: true
Attachment #353792 -
Attachment is obsolete: true
Assignee | ||
Comment 7•16 years ago
|
||
This patch can be checked in prior to the change to Unicode NSIS.
Attachment #354025 -
Attachment is obsolete: true
Attachment #354026 -
Flags: review?(philringnalda)
Assignee | ||
Comment 8•16 years ago
|
||
bah... I forgot to include the change to defines.nsi.in Once again, this patch can be checked in prior to the change to Unicode NSIS.
Attachment #354026 -
Attachment is obsolete: true
Attachment #354027 -
Flags: review?(philringnalda)
Attachment #354026 -
Flags: review?(philringnalda)
Assignee | ||
Comment 9•16 years ago
|
||
This patch requires the patch from bug 305039 and won't be pushed to comm-central until the patch from bug 305039 is pushed to mozilla-1.9.1
Attachment #354094 -
Flags: review?(philringnalda)
Updated•16 years ago
|
Attachment #354027 -
Flags: review?(philringnalda) → review+
Comment 10•16 years ago
|
||
Comment on attachment 354027 [details] [diff] [review] patch for locale and general cleanup rev3 - checked in r=me, thanks! (Amazingly difficult to actually look at strings in the installer wizard, after thousands of times rushing through as fast as possible - I almost resorted to taking screenshots so I couldn't automatically click Next.)
Comment 11•16 years ago
|
||
Comment on attachment 354094 [details] [diff] [review] patch for Unicode support rev2 (checked in) I'm not going to pretend to actually understand it all, but it built a perfectly fine en-US installer for me, so Looks Good To Me :)
Attachment #354094 -
Flags: review?(philringnalda) → review+
Assignee | ||
Comment 12•16 years ago
|
||
Comment on attachment 354027 [details] [diff] [review] patch for locale and general cleanup rev3 - checked in Pushed to comm-central http://hg.mozilla.org/comm-central/rev/82c571b8e4b3
Attachment #354027 -
Attachment description: patch for locale and general cleanup rev3 → patch for locale and general cleanup rev3 - checked in
Assignee | ||
Comment 13•16 years ago
|
||
Thanks Phil Leaving open to land the patch for Unicode support after bug 305039 lands
Assignee | ||
Comment 14•15 years ago
|
||
Pushed to comm-central http://hg.mozilla.org/comm-central/rev/7ae154a2a598
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b2
Assignee | ||
Updated•15 years ago
|
Attachment #354094 -
Attachment description: patch for Unicode support rev2 → patch for Unicode support rev2 (checked in)
Updated•15 years ago
|
Summary: Update installer for Unicode support and locale verification → Update Thunderbird installer for Unicode support and locale verification
You need to log in
before you can comment on or make changes to this bug.
Description
•