Closed
Bug 347087
Opened 18 years ago
Closed 18 years ago
Installer doesn't remove old files from components/ on upgrade
Categories
(Calendar :: Sunbird Only, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mattwillis, Assigned: mattwillis)
Details
(Whiteboard: [patch in hand])
Attachments
(1 file)
|
1.51 KB,
patch
|
dmosedale
:
first-review+
|
Details | Diff | Splinter Review |
In bug 320366, we (correctly) moved all files that didn't implement nsIModule from inside the $mozappdir/components/ directory. dmose noticed that when upgrading on Windows using the NSIS installer, we didn't REMOVE those old files. We need to do that, and also need to be careful in the future so we know what files we'll want to delete on upgrade moving forward.
|
Assignee | |
Comment 1•18 years ago
|
||
This patch adds the files that were moved to the "removed-files.in" list. When upgrading, we'll delete those files if found.
Attachment #231850 -
Flags: first-review?(dmose)
|
|
Assignee | |
Comment 2•18 years ago
|
||
This will cause all sorts of problems on upgrade if not fixed before release Nominating for blocking0.3
Status: NEW → ASSIGNED
Flags: blocking0.3?
Whiteboard: [patch in hand]
|
||
Updated•18 years ago
|
Flags: blocking0.3? → blocking0.3+
|
||
Comment 3•18 years ago
|
||
Comment on attachment 231850 [details] [diff] [review] Adds files formerly in components/ to removed-files.in r=dmose
Attachment #231850 -
Flags: first-review?(dmose) → first-review+
|
|
Assignee | |
Comment 4•18 years ago
|
||
Patch checked in on branch and trunk -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•