Closed Bug 1134913 Opened 8 years ago Closed 8 years ago

Disable EME voucher generation on Win64


(Release Engineering :: General, defect)

Windows 8
Not set


(firefox37 fixed, firefox38 fixed, firefox39 fixed)

Tracking Status
firefox37 --- fixed
firefox38 --- fixed
firefox39 --- fixed


(Reporter: cpearce, Assigned: cpearce)


(Blocks 1 open bug)



(1 file)

Bug 1131798 was backed out because the patch to the EME voucher generator can't handle Win64 binaries. We need bug 1131798 urgently, so we should just disable running the voucher generator on Win64 so that bug 1131798 can land, and follow up to re-enable it on Win64.

We should only be running the on 32bit Windows builds.
That looks like the place.
Attached patch PatchSplinter Review
Don't run pn Win64, since it isn't designed to handle it anyway.

Try looks good.

Thanks for the tip Rail.
Attachment #8567004 - Flags: review?(ted)
Yay, no problem.
Attachment #8567004 - Flags: review?(ted) → review+
Please make sure to file a follow-up and have it block bug 558448?
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #6)
> Please make sure to file a follow-up and have it block bug 558448?

Filed bug 1135320.
Assignee: nobody → cpearce
Closed: 8 years ago
Resolution: --- → FIXED
Needs uplift.
Flags: needinfo?(cpearce)
Comment on attachment 8567004 [details] [diff] [review]

Approval Request Comment
[Feature/regressing bug #]: EME
[User impact if declined]: We'll break the build when we land Bug 1131798, which is required for Adobe EME to work.
[Describe test coverage new/current, TreeHerder]: We haven't yet setup an automated test yet as this requires downloading Adobe's CDM over the internet, and our existing test harnesses can't do that. We're going to setup an external test that covers this.
[Risks and why]: Low; only touches affects third party plugins.
[String/UUID change made/needed]: None
Attachment #8567004 - Flags: approval-mozilla-beta?
Comment on attachment 8567004 [details] [diff] [review]

Taking this as a prereq for bug 1131798. Beta+
Attachment #8567004 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: needinfo?(cpearce)
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.