Last Comment Bug 495232 - Drop support for the Palm Sync Extension
: Drop support for the Palm Sync Extension
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All Windows XP
: -- normal (vote)
: Thunderbird 3.0b3
Assigned To: Mark Banner (:standard8)
:
Mentors:
http://ccgi.standard8.plus.com/blog/a...
Depends on:
Blocks: 496281
  Show dependency treegraph
 
Reported: 2009-05-28 07:17 PDT by Mark Banner (:standard8)
Modified: 2009-06-04 01:29 PDT (History)
7 users (show)
standard8: in‑litmus-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
mailnews/ patch (364.90 KB, patch)
2009-05-28 07:29 PDT, Mark Banner (:standard8)
no flags Details | Diff | Review
mailnews/ patch (356.90 KB, patch)
2009-05-28 07:31 PDT, Mark Banner (:standard8)
mnyromyr: review+
mozilla: superreview+
Details | Diff | Review
mail/ patch (960 bytes, patch)
2009-05-28 07:33 PDT, Mark Banner (:standard8)
mozilla: review+
Details | Diff | Review
suite/ patch (7.36 KB, patch)
2009-05-28 07:35 PDT, Mark Banner (:standard8)
bugzilla: review+
Details | Diff | Review

Description Mark Banner (:standard8) 2009-05-28 07:17:55 PDT
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.
Comment 1 Mark Banner (:standard8) 2009-05-28 07:29:57 PDT
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.
Comment 2 Mark Banner (:standard8) 2009-05-28 07:31:44 PDT
Created attachment 380136 [details] [diff] [review]
mailnews/ patch

The correct patch this time...
Comment 3 Mark Banner (:standard8) 2009-05-28 07:33:36 PDT
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.
Comment 4 Mark Banner (:standard8) 2009-05-28 07:35:42 PDT
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.

Note You need to log in before you can comment on or make changes to this bug.