841636 - Temporarily disable WebRTC in SeaMonkey build to work around bug 837618

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Edmund Wong (:ewong)]

+++ This bug was initially created as a clone of Bug #839046 +++

Bug 837618 is a problem where the list of file names for object files to be linked together is too long for the linker to handle. It can be fixed in clang, but in the meantime our Mac nightlies are broken.

I'm proposing that we temporarily disable WebRTC (which has plenty of object files should give us some breathing space), and in actual fact is not correctly packaged at the moment anyway, also on top of that argument, is that WebRTC hasn't yet been fully enabled in trunk.

Once bug 837618 is fixed, then we can re-enable WebRTC properly with the appropriate packaging etc.

Comment 1

[Edmund Wong (:ewong)]

Attachment: Temporarily disable WebRTC. (v1)

Comment 2

[Justin Wood (:Callek)]

Comment on attachment 714215 [details] [diff] [review]
Temporarily disable WebRTC. (v1)

Review of attachment 714215 [details] [diff] [review]:
-----------------------------------------------------------------

Patch is technically sound, but on the matter of "Do we want it" I'm unsure when I can get around to looking deep enough to make the call. I know me and Ratty were discussing this recently. So if he feels we should, happy to have him mark this r+ for me.

Comment 3

[Philip Chee]

Comment on attachment 714215 [details] [diff] [review]
Temporarily disable WebRTC. (v1)

f=me but Callek will make the final decision.

Comment 4

[Philip Chee]

Comment on attachment 714215 [details] [diff] [review]
Temporarily disable WebRTC. (v1)

After discussion on IRC, I'm changing my f+ to r+ to get our OSX buildbots building again.

Comment 5

[Edmund Wong (:ewong)]

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/a0b3fd5a41f3

Comment 6

[Philip Chee]

symbols.def : error LNK2001: unresolved external symbol vpx_codec_vp8_cx
gkmedias.lib : fatal error LNK1120: 1 unresolved externals

Something seems to have regressed Bug 799069 - Unresolved vpx_codec_vp8_cx export in --disable-webrtc build

Or perhaps I just need a clobber?

Comment 7

[Justin Wood (:Callek)]

Comment on attachment 714215 [details] [diff] [review]
Temporarily disable WebRTC. (v1)

Review of attachment 714215 [details] [diff] [review]:
-----------------------------------------------------------------

Ratty took care of this review :-)

Comment 8

[Edmund Wong (:ewong)]

AIUI, this was backed out a while, so I'm closing this.