Closed Bug 794412 Opened 7 years ago Closed 7 years ago

Update gonk-toolchain to support building OMX-in-gecko


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: cjones, Assigned: cjones)




(2 files)

I thought this would be trivial but it's turning into a tar baby.  Fighting some C++ includes right now.

The toolchain produced from this work won't be publicly publishable (sigh), so we'll have to choose how the bits are pushed around.  It's fine for them to live on mozilla servers that aren't publicly accessibly (like OS X, windows, etc.).
Could do a PR, but this is mostly to exchange information for now.  This could land upstream a month from now.
Assignee: nobody → jones.chris.g
Attachment #664893 - Flags: review?(jhford)
The includes from stlport/ conflict with some from ndk/.  We need to include some of the stlport dirs directly, but we also need jni.h for the omx code here, which is brought in through ndk/.  It's not wise to let ndk and stlport fight over their headers, so instead we just directly include what we need.  There's no reason a priori that the stlport directory layout would change more than the ndk's.

This patch should allow both versions of the toolchain to work until we move to the new one.
Attachment #664901 - Flags: review?(chris.double)
Attachment #664901 - Flags: review?(chris.double) → review+
Could you explain what this bug actually means?
We need to build a new toolchain to support building OMX-decoder-built-into-gecko (bug 759506).  Unfortunately, this also requires a gecko patch.
Blocks: 759506
Attachment #664893 - Flags: review?(jhford) → review+
I confirmed with the patch here, m-c builds without bug 759506, and with the new toolchain applied it also builds.
Pointing at new b2g-toolchain on mozilla-central:

Green on try
Closed: 7 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.