dom/media/platforms/gonk/GonkDecoderModule.cpp:43:12: error: 'kNeedAnnexB' was not declared in this scope

RESOLVED FIXED

Status

Firefox OS
GonkIntegration
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: gerard, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(firefox51 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

Comment hidden (mozreview-request)

Comment 2

a year ago
mozreview-review
Comment on attachment 8782451 [details]
Bug 1296295 - Followup bug 1295920, replace enum with enum class

https://reviewboard.mozilla.org/r/72622/#review70242
Attachment #8782451 - Flags: review?(jyavenard) → review+

Comment 3

a year ago
Pushed by alissy@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5722cb5f4d22
Followup bug 1295920, replace enum with enum class r=jya
Hi Alexandre,

Could you please tell me how to ensure the tier3 platform can be built on try server?

I had pushed a try to validate all the platforms can pass the build [1], but it seems the error log from tier3 will be various in different errors... it is hard to notice the build fail case which is due to the patch we provide.

Is there any good way to do ensure this? 


Thank you and thanks for fixing this!

[1]https://treeherder.mozilla.org/#/jobs?repo=try&revision=b8996c98fd88&filter-tier=1&filter-tier=2&filter-tier=3
Flags: needinfo?(lissyx+mozillians)
(Reporter)

Comment 5

a year ago
Well, your link *should* have been good. Now, being tier 3 means we get broken more easily. In this case, what happens is that we thought we moved everything away from git.mozilla.org which was decommissionned a couple of days ago. Turns out we forgot a couple of references to it, and this broke builds. Hence the failure we can see in your log. Generally speaking, your approach should have worked. In this case, however, bad luck came into play :/

But thanks for trying to not break this!
Flags: needinfo?(lissyx+mozillians)

Comment 6

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5722cb5f4d22
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.