386908 - remove xpfe/communicator from Thunderbird build

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Robert Kaiser]

Thunderbird currently still goes into xpfe/communicator in its build, see http://mxr.mozilla.org/seamonkey/source/xpfe/Makefile.in#53

We should get rid of this so that xpfe/communicator can ultimately be removed from the source (SeaMonkey uses its own version from suite/ anyways).

While at this, we should probably look into the places where we are using communicator files in Thunderbird as well, as I guess that http://mxr.mozilla.org/seamonkey/source/mail/base/jar.mn#133 (and 134) don't actually work, as communicator is probably not registered in toolkit.jar but only in comm.jar so those two files are probably not actually used anywhere from that place.

Comment 1

[Robert Kaiser]

I just did a build of Thunderbird without entering xpfe/components and the two toolkit.jar files removed in the location pointed out above, and everything seemed to work (once I added registration of the communicator chrome package to mail/bar/jar.mn).

I'll file a patch tomorrow, this should make everything a lot cleaner :)

Comment 2

[Robert Kaiser]

Attachment: don't build xpfe/communicator for Thunderbird

This is the patch for what I mentioned above - the only thing Thunderbird still needs from xpfe/communicator is the one line that's registering the communicator chrome package, and that can easily be copied to mail/base (which I'm doing here).
Testing a build with this patch shows no errors in chrome or error console and everything seems to be working fine.

Comment 3

[Phil Ringnalda (:philor)]

I don't suppose you own a printer? It looks to me like you're breaking printing, since we include chrome://communicator/content/printing.js several places that would need chrome://global/content/printUtils.js instead. (And I really, really hope it's a drop-in replacement.)

Comment 4

[Robert Kaiser]

Interesting. I don't know printing too well, but as a toolkit app, it's surely better to use a toolkit file than a communicator file. printUtils rev. 1.1 says "add printUtils.js to the toolkit (copied from printing.js)" in the checkin comment, so it actually should be exchangeable (oh, and if it's referenced in files shared with SeaMonkey, we should also change it, as SeaMonkey uses toolkit as well now).

Comment 5

[Phil Ringnalda (:philor)]

Comment on attachment 271023 [details] [diff] [review]
don't build xpfe/communicator for Thunderbird

I've been running with this patch so long waiting for the printing reviews to work their way up the chain, I had to look it up earlier today to see what it was, so it ought to be pretty safe :)

r=philringnalda, and thanks!

Comment 6

[Steffen Wilberg]

So if we don't ship this 0-byte file we can probably cvs remove it?
http://mxr.mozilla.org/seamonkey/source/mail/base/content/tasksOverlay.xul
The checkin comment was "empty out tasksOverlay.xul. We don't use it anymore but someone else (in a file we don't fork) may try to load it."

Comment 7

[Robert Kaiser]

Steffen:
That's why I haven't marked this bug FIXED yet ;-)

Comment 8

[Robert Kaiser]

OK, checked in, everything's green, and r/sr/agreement on IRC from Phil and mscott for the CVS removal of tasksOverlay.xul