614205 - Drop unnecessary NECKO_PROTOCOLS_DEFAULT in confvars.sh

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

http://mxr.mozilla.org/comm-central/search?string=NECKO_PROTOCOLS_DEFAULT

iirc we used to specify this so we didn't build in things like gopher. However that changed, and there's nothing specific we don't want. In addition, the recently added wyciwyg is not getting built into the build, and this breaks IPC builds of Thunderbird (which I'm working towards).

So I think we should just remove it and use the same that mozilla-central uses.

Also, I'm removing the define for Sunbird just to get it out of the way when searching.

Comment 1

[Justin Wood (:Callek)]

Comment on attachment 492606 [details] [diff] [review]
The fix

Code-Change is good. I don't consider myself qualified [with TB team/plans] on the "do we want this" but you are, and I have no objections based on your reasoning.

Comment 2

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/b91fd1c9b44e