Closed Bug 1247175 Opened 4 years ago Closed 4 years ago

libavcodec.so.57 not supported when micro version is > 100

Categories

(Core :: Audio/Video: Playback, defect)

46 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox45 --- fixed
firefox46 + fixed
firefox47 + fixed

People

(Reporter: gerald, Assigned: gerald)

References

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

FFmpeg's libavcodec's micro version has recently been bumped from 100 to 101.
Due to a misunderstanding, we assumed that the micro version (of version 57) would always remain exactly 100:
http://mxr.mozilla.org/mozilla-central/source/dom/media/platforms/ffmpeg/FFmpegLibWrapper.cpp#43

The test should be relaxed to allow micro versions greater or equal to 100.

This code dates from bug 1232268, which landed in FF46, so we should uplift a fix to Aurora-46 if possible.
Attachment #8717788 - Flags: review?(jyavenard) → review+
Comment on attachment 8717788 [details]
MozReview Request: Bug 1247175 - Accept libavcodec 57.100+ - r?jya

https://reviewboard.mozilla.org/r/34323/#review31011
NI:myself to request uplift to Aurora-46.
Flags: needinfo?(gsquelart)
https://hg.mozilla.org/mozilla-central/rev/1568bce05431
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee: nobody → gsquelart
Flags: needinfo?(gsquelart)
Approval Request Comment
[Feature/regressing bug #]: MP4 playback on Linux using FFmpeg
[User impact if declined]: MP4 playback would stop working if users upgrade their install of FFmpeg (directly, or indirectly through OS upgrades).
[Describe test coverage new/current, TreeHerder]: Lots of media tests, 2 days in Nightly, try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=79f3af745bed
[Risks and why]: Tiny change ('!=100' -> '<100') to allow future minor versions of FFmpeg 57, so I don't see any risks.
[String/UUID change made/needed]: None.
Attachment #8717788 - Attachment is obsolete: true
Attachment #8719218 - Flags: review+
Attachment #8719218 - Flags: approval-mozilla-aurora?
As per comment #0, the issue is with code that landed in FF46 (currently Aurora), see bug 1232268.
Recent regression in 46, tracking.
Keywords: regression
Comment on attachment 8719218 [details] [diff] [review]
1247175-accept-libavcodec-57-101-plus.patch

Fix for recent regression, try push ok. Please uplift to aurora.
Attachment #8719218 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 8719218 [details] [diff] [review]
1247175-accept-libavcodec-57-101-plus.patch

Should patch 1232268 be approved, we always need this one (though it will need rebasing, still two characters change)

Approval Request Comment
[Feature/regressing bug #]: 1247175
[User impact if declined]: New LTS version of various distributions will ship with FFmpeg 3.0. ESR45 will not be usable on those systems.
[Describe test coverage new/current, TreeHerder]: in central for a few weeks.
[Risks and why]: Low, it's well tested and there isn't much to it.
[String/UUID change made/needed]: None

This means ESR45 won't be able to play mp4, mp3, aac etc...
Attachment #8719218 - Flags: approval-mozilla-beta?
Comment on attachment 8719218 [details] [diff] [review]
1247175-accept-libavcodec-57-101-plus.patch

Want that in 45 too, taking & should be in 45 beta 10
Attachment #8719218 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
this bugs only make sense if bug 1232268 is approved for beta.
(In reply to Jean-Yves Avenard [:jya] from comment #13)
> this bugs only make sense if bug 1232268 is approved for beta.

holding since bug 1232268  has conflicts
Flags: needinfo?(gsquelart)
Thank you Carsten and Jean-Yves.
https://hg.mozilla.org/releases/mozilla-beta/rev/54424f42b757
Flags: needinfo?(gsquelart)
You need to log in before you can comment on or make changes to this bug.