Closed
Bug 368401
Opened 18 years ago
Closed 18 years ago
Installer does not delete myspell directory
Categories
(SeaMonkey :: Installer, defect)
SeaMonkey
Installer
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ok34, Assigned: ajschult784)
Details
(Keywords: fixed-seamonkey1.1.1)
Attachments
(1 file)
2.90 KB,
patch
|
neil
:
review+
csthomas
:
approval-seamonkey1.1.1+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.8.1.2pre) Gecko/20070118 SeaMonkey/1.1 Build Identifier: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.8.1.2pre) Gecko/20070118 SeaMonkey/1.1 When migrating from Seamonkey 1.07 to 1.1 using the installer, the myspell directory does not get deleted. Reproducible: Always Steps to Reproduce: 1. Upgrade from SM 1.07 to 1.1 2. Look into [install dir]\components\myspell Actual Results: Directory and content exists Expected Results: Should IMHO have been deleted (a clean install of SM 1.1. does not create a myspell directory). Maybe some contents can or shall be retained, e.g. personal spelling preferences and additions. This possibly exists on other platforms as well, I just haven't had the time to check on Linux or Windows.
Comment 1•18 years ago
|
||
Probably similar to what we just did in bug 367657 some deleteThisFolder() / deleteThisFile() lines have to be added to browser.jst. Or would we have to move the existing files from components/myspell/ to dictionaries/ before removing the old directory, in case the user wants to keep what is there? I can confirm the problem on OS/2, but not sure if this is specific to OS/2, either. CCing some people who might know from the work in bug 350688.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 2•18 years ago
|
||
there is no move, although it could be added. Current methods are implemented in http://mxr.mozilla.org/seamonkey/source/xpinstall/packager/common/share.t This deletes myspell on windows & OS/2 and also rearranges OS/2 a bit in an attempt at coherency. Unix's browser.jst is woefully out of date and irrelevant anyway (the installer nukes the old directory before installing), so I feel unmotivated to "maintain" that list.
Assignee | ||
Updated•18 years ago
|
OS: OS/2 → All
Hardware: Other → All
Version: unspecified → Trunk
Comment 3•18 years ago
|
||
Comment on attachment 253018 [details] [diff] [review] delete myspell sr=me
Attachment #253018 -
Flags: review?(neil) → review+
Assignee | ||
Comment 4•18 years ago
|
||
Comment on attachment 253018 [details] [diff] [review] delete myspell (landed on trunk) needed on the branch too
Attachment #253018 -
Flags: approval-seamonkey1.1.1?
Assignee | ||
Updated•18 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment 5•18 years ago
|
||
Comment on attachment 253018 [details] [diff] [review] delete myspell first-a=me for 1.1.1, still one needed to go
Comment on attachment 253018 [details] [diff] [review] delete myspell a=me for sm1.1.1
Attachment #253018 -
Flags: approval-seamonkey1.1.1? → approval-seamonkey1.1.1+
Assignee | ||
Updated•17 years ago
|
Keywords: fixed-seamonkey1.1.1
You need to log in
before you can comment on or make changes to this bug.
Description
•