Closed
Bug 236923
Opened 21 years ago
Closed 21 years ago
palm sync uninstall doesn't always restore previous adress conduit
Categories
(MailNews Core Graveyard :: Palm Sync, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
Details
Attachments
(1 file, 1 obsolete file)
18.53 KB,
patch
|
mscott
:
superreview+
|
Details | Diff | Splinter Review |
The palm sync uninstall attempts to re-install the desktop ab conduit by calling
CmRestoreHotSyncSettings, which is supposed to restore the conduit settings to
their default values. This has a bunch of problems, including: if the palm
desktop isn't in the default location, uninstall doesn't work, and it doesn't
restore the previous conduit, just the default desktop conduit. A better
approach is to save off the previous conduit information in the registry at
install time, and restore it at uninstall time. Patch upcoming.
Assignee | ||
Comment 1•21 years ago
|
||
this fix points out the need to cleanup the code that gets the function
pointers from the conduit mgr - this should be table driven...
Assignee | ||
Comment 2•21 years ago
|
||
removed unneeded changes to GetPalmDesktopInstallDirectory
Attachment #143406 -
Attachment is obsolete: true
Assignee | ||
Updated•21 years ago
|
Attachment #143411 -
Flags: superreview?(mscott)
Updated•21 years ago
|
Attachment #143411 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 3•21 years ago
|
||
fixed on trunk (previously fixed on branch)
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: MailNews → Core
Comment 4•20 years ago
|
||
*** Bug 219227 has been marked as a duplicate of this bug. ***
Comment 5•19 years ago
|
||
*** Bug 238329 has been marked as a duplicate of this bug. ***
Comment 6•19 years ago
|
||
*** Bug 201661 has been marked as a duplicate of this bug. ***
Comment 7•19 years ago
|
||
*** Bug 199729 has been marked as a duplicate of this bug. ***
Updated•19 years ago
|
QA Contact: nbaca → vseerror
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•