[Win32] Installer should remove Thunderbird 3 files during downgrade to avoid potential crashes

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
10 years ago
7 years ago

People

(Reporter: rstrong, Unassigned)

Tracking

x86
Windows Vista

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [notacrash])

Attachments

(1 attachment)

Spinoff of bug 423226 for Thunderbird. All files that exist in a Thunderbird 3 install that don't exist in a Thunderbird 2 install should be added to removed-files.in
Serge, you probably want this for SeaMonkey too.
Blocks: 450504
Thanks, I filed bug 450504 ;-)
Created attachment 344244 [details] [diff] [review]
Untested patch

Not sure what to do with this. This is a list from following the steps in bug 423226 (install Tb 2.0.0.17, set the file URL for a saved copy of attachment 324105 [details] as the start page, "List Files 1", install Tb 3.0a3 over the top, install Tb 2.0.0.17 over the top, "List Files 2"), but thanks to header and compiler bingo, I no longer have a build environment for 1.8 on Windows, so I can't actually test whether a build-with-my-patch, trunk, build-with-my-patch comes out clean.
I will probably be setting up a 1.8 environment in the next few days/week, so will be able to test for you then.
Comment on attachment 344244 [details] [diff] [review]
Untested patch

Thanks Phil. The file path delimiter should be forward slashes and not backslashes... it also appears that 
modules/WindowDraggingUtils.jsm is missing.
(In reply to comment #5)
> modules/WindowDraggingUtils.jsm is missing.

Yeah, that was another component of "Not sure what to do with this." - I built my list from the Tb3.0a3 release, because that's the only released thing where I'll feel sorry for someone who gets into overwrite trouble, but it's too late for 2.0.0.18 already, but is 3.0b1 going to either ship or code freeze before the code freeze for .19 ... we don't really have a good process for handling "any new file created in a trunk nightly must be listed in a file on the stable branch." (Hmm, how about nsFolderLookupService.js that we shipped for a week in early September?)

Updated

10 years ago
Summary: [Win32] Installer should remove Thunderbird 3 files during downgrade to avoid crash on start → [Win32] Installer should remove Thunderbird 3 files during downgrade to avoid crash on start [@ nsACString_internal::Assign][@ xpcom_core.dll]
Wayne: did you actually crash there downgrading, or just copy that change in summary over from the Firefox bug? That particular nsACString_internal::Assign crash had a stack running through brwsrcmp.dll, and was presumably the Firefox 3 feed sniffer doing bad things in a Firefox 2.
Summary: [Win32] Installer should remove Thunderbird 3 files during downgrade to avoid crash on start [@ nsACString_internal::Assign][@ xpcom_core.dll] → [Win32] Installer should remove Thunderbird 3 files during downgrade to avoid potential crashes

Comment 8

9 years ago
philor I no longer remember. I didn't record that I got that crash myself (and crash stats is useless for checking that time period), so I would hope that I had checked crash stats.  However, it's quite possible I copied it from FF.

I am fine with what you feel is most accurate for the summary
Whiteboard: [notacrash]

Comment 9

9 years ago
do we have a "flip the update from v2" bug that this should block?
or is there thinking this is no longer needed?

Comment 10

7 years ago
Is this still an issue? SeaMonkey bug 450504 has been won't-fixed given that the old branches are no longer supported.
Yeah, I don't think we need this one now.
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.