Drop support for the Palm Sync Extension

RESOLVED FIXED in Thunderbird 3.0b3

Status

MailNews Core
Build Config
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Thunderbird 3.0b3
All
Windows XP
Bug Flags:
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

8 years ago
It has been a few days since I posted, and I've not heard any negative comments about dropping support for Palm Sync, so I think its time to do it.

I have archived the code under a repo in my users directory:

http://hg.mozilla.org/users/bugzilla_standard8.plus.com/palmsync/

and added a readme.txt with some information about it.

I'll be filing a separate bug on archiving the palm sync component.
(Assignee)

Comment 1

8 years ago
Created attachment 380134 [details] [diff] [review]
mailnews/ patch

This drops palm sync from within mailnews. A couple of notes:

- The do extensions directory after everything else hack is no longer needed.
- Address book no longer needs to export nsAbCardProperty.h; it was only palm sync that required the export.
- Majority of this patch is the file removals.
Attachment #380134 - Flags: superreview?(bienvenu)
Attachment #380134 - Flags: review?(mnyromyr)
(Assignee)

Comment 2

8 years ago
Created attachment 380136 [details] [diff] [review]
mailnews/ patch

The correct patch this time...
Attachment #380134 - Attachment is obsolete: true
Attachment #380136 - Flags: superreview?(bienvenu)
Attachment #380136 - Flags: review?(mnyromyr)
Attachment #380134 - Flags: superreview?(bienvenu)
Attachment #380134 - Flags: review?(mnyromyr)
(Assignee)

Comment 3

8 years ago
Created attachment 380137 [details] [diff] [review]
mail/ patch

mail/ removals - we no longer need to worry about not packaging PalmSyncInstall.exe, and the prefs are not a concern either now.
Attachment #380137 - Flags: review?(bienvenu)
(Assignee)

Comment 4

8 years ago
Created attachment 380138 [details] [diff] [review]
suite/ patch

Removes palm sync from the SeaMonkey installer. Thankfully this is quite easy as I made all the field positions automatically work themselves out.
Attachment #380138 - Flags: review?
(Assignee)

Updated

8 years ago
Attachment #380138 - Flags: review? → review?(bugzilla)

Updated

8 years ago
Attachment #380137 - Flags: review?(bienvenu) → review+

Updated

8 years ago
Attachment #380136 - Flags: superreview?(bienvenu) → superreview+

Updated

8 years ago
Attachment #380138 - Flags: review?(bugzilla) → review+

Updated

8 years ago
Attachment #380136 - Flags: review?(mnyromyr) → review+
(Assignee)

Updated

8 years ago
Blocks: 496281
(Assignee)

Comment 5

8 years ago
All patches checked in: 

http://hg.mozilla.org/comm-central/rev/2c2aabae129f
http://hg.mozilla.org/comm-central/rev/3c14e195490c
http://hg.mozilla.org/comm-central/rev/dd584408d621
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Flags: in-litmus-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.