1533598 - flac not supported on Windows on ARM64

Status: NEW

(Core :: Audio/Video: Playback, enhancement, P3)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

Discovered in bug 1526080 comment 3, we're not building ffvpx on aarch64 Windows builds, and so flac isn't supported there.

I guess we're not doing the right thing here:

https://searchfox.org/mozilla-central/rev/b2d35912da5b2acecb0274eb113777d344ffb75e/toolkit/moz.configure#1554

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Mike: Do you have an opinion on the moz.configure line referenced in comment 0 above? We want flac support on Windows on ARM64, is there a reason why Windows+aarch64 is excluded in the configure logic linked to above?

Comment 2

[Mike Hommey [:glandium]]

Jean-Yves might know?

Comment 3

[Jean-Yves Avenard [:jya]]

it was removed in bug 1481513.

On arm (be it 32 or 64 bits) we only compile ffvpx in plain C. Should have compiled just fine on arm64

Comment 4

[Nathan Froyd [:froydnj]]

ffvpx's "plain C" was really "plain C assuming a bunch of Linux-isms and/or GCC-isms" and there was zero aarch64 windows support.

Comment 5

[Jean-Yves Avenard [:jya]]

(In reply to Nathan Froyd [:froydnj] from comment #4)

ffvpx's "plain C" was really "plain C assuming a bunch of Linux-isms and/or GCC-isms" and there was zero aarch64 windows support.

I feel that my earlier comment may have been misinterpreted. There was no intention to blame or put criticism. I was just stating my recollection on event on what led flac to no longer be compiled for arm64.