User:Florian/building purplexpcom for Thunderbird: Difference between revisions

Jump to navigation Jump to search
no edit summary
mNo edit summary
No edit summary
Line 7: Line 7:
The changesets I pushed to the try server contain:
The changesets I pushed to the try server contain:
* the purple/ folder of the Instantbird repository (any revision that contains the patch from [https://bugzilla.instantbird.org/show_bug.cgi?id=1665 Instantbird bug 1665] and its follow ups should work). Let's say I used the [http://hg.instantbird.org/instantbird/file/29d2a239ba24/purple purple/ folder of revision 29d2a239ba24].
* the purple/ folder of the Instantbird repository (any revision that contains the patch from [https://bugzilla.instantbird.org/show_bug.cgi?id=1665 Instantbird bug 1665] and its follow ups should work). Let's say I used the [http://hg.instantbird.org/instantbird/file/29d2a239ba24/purple purple/ folder of revision 29d2a239ba24].
* a patch to adapt the comm-central build system to build purple/
* a patch to adapt the comm-central build system to build purple/ (Thunderbird [http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/build-system-tb15.patch 15], [http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/build-system-tb16.patch 16] and [http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/build-system-tb17.patch 17] versions)
* some very simple changes to purple/ to disable the jabber and qq libpurple protocol plugins (jabber because Thunderbird already ships by default a generic XMPP plugin and I don't want to override it, and qq because it doesn't work correctly at the moment, even for Instantbird), and to disable purple/'s xpcshell tests (I didn't request tests on the try server runs, so just commenting out the lines related to the test in the purple/ Makefile was easier than adding a line in mail/'s xpcshell.ini file to make the harness find them).
* some very simple changes to purple/ to disable the jabber and qq libpurple protocol plugins (jabber because Thunderbird already ships by default a generic XMPP plugin and I don't want to override it, and qq because it doesn't work correctly at the moment, even for Instantbird), and to disable purple/'s xpcshell tests (I didn't request tests on the try server runs, so just commenting out the lines related to the test in the purple/ Makefile was easier than adding a line in mail/'s xpcshell.ini file to make the harness find them). ([http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/purple-changes-tb15.patch patch available here])
* when building for Thunderbird 16 and 17, some fixes to keep purplexpcom building against newer gecko versions. For Thunderbird 16 I had deal with the removal of nsILocalFile, and for Thunderbird 17 I also had to deal with the nsnull->nullptr change and the removal of the NS_OUTPARAM and NS_SCRIPTABLE macros (this patch has been posted to [https://bugzilla.instantbird.org/show_bug.cgi?id=1606 Instantbird bug 1606] and reviewed there.
* when building for Thunderbird 16 and 17, the same patch has some additional fixes to keep purplexpcom building against newer gecko versions. For Thunderbird 16 I had deal with the removal of nsILocalFile, and for Thunderbird 17 I also had to deal with the nsnull->nullptr change and the removal of the NS_OUTPARAM and NS_SCRIPTABLE macros (this patch has been posted to [https://bugzilla.instantbird.org/show_bug.cgi?id=1606 Instantbird bug 1606] and reviewed there. (Patch available here for the [http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/purple-changes-tb16.patch Thunderbird 16] and [http://ftp.instantbird.com/instantbird/thunderbird-addon/2.10.4/diffs/purple-changes-tb17.patch Thunderbird 17] versions)


I included in the add-on localized strings for the top 4 locales of Thunderbird (en-US, de, fr, ja) from http://hg.instantbird.org/l10n/libpurple/
I included in the add-on localized strings for the top 4 locales of Thunderbird (en-US, de, fr, ja) from http://hg.instantbird.org/l10n/libpurple/
Confirmed users
87

edits

Navigation menu