Confirmed users
87
edits
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/ |