Closed Bug 933320 Opened 8 years ago Closed 8 years ago

properly cross-compile WebRTC


(Core :: WebRTC, defect)

Not set





(Reporter: froydnj, Assigned: froydnj)



(Whiteboard: [qa-])


(2 files, 3 obsolete files)

Over in bug 921040, we're trying to compile Firefox for Mac on a Linux host.  There's no special logic in the Mozilla build system to tell WebRTC that we're cross-compiling.

I think I have this nailed for the darwin-cross case; I'll have to make the patch more general before it's ready, though.
This part doesn't work if you don't specify --with-macos-sdk on the configure
line.  I'm not sure how to deal with that, and I am seeking feedback for how
to bend gyp to my will.

Nominating Ted, because he might understand gyp.
Attachment #825389 - Flags: feedback?(ted)
Let's not assume host==target when compiling WebRTC.
Attachment #825390 - Flags: review?(ted)
Let's not assume host==target and let's not assume we know where our sdk is.
Attachment #825390 - Attachment is obsolete: true
Attachment #825390 - Flags: review?(ted)
Attachment #825391 - Flags: review?(ted)
Attachment #825391 - Flags: review?(ted) → review+
Turns out we don't have to do any violence to the gyp files.  We just have to
make not complain and silently return a bogus valuein such cases.
People compiling for OS X and not setting --with-macos-sdk get what they
Attachment #825389 - Attachment is obsolete: true
Attachment #825389 - Flags: feedback?(ted)
Attachment #828708 - Flags: review?(ted)
Same patch as before, but using the "correct" mac_sdk_path gyp variable, rather than
the subtly different mac_sdk gyp variable.  No functional change.
Attachment #825391 - Attachment is obsolete: true
Attachment #828709 - Flags: review?(ted)
Comment on attachment 828709 [details] [diff] [review]
part 2 - inform WebRTC of our chosen target when invoking gyp

Whoops, meant to carry over ted's r+ on this one.
Attachment #828709 - Flags: review?(ted) → review+
Comment on attachment 828708 [details] [diff] [review]
part 1 - make silently comply if we're not running on a Mac host

Review of attachment 828708 [details] [diff] [review]:

This probably needs to go upstream.
Attachment #828708 - Flags: review?(ted) → review+
Assignee: nobody → nfroyd
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
hmm this broke webrtc compilation on linux arm target... is anything need to be done to make it works back?
Depends on: 937990
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.