Closed Bug 1247175 Opened 7 years ago Closed 7 years ago not supported when micro version is > 100


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

46 Branch
Not set



Tracking Status
firefox45 --- fixed
firefox46 + fixed
firefox47 + fixed


(Reporter: mozbugz, Assigned: mozbugz)



(Keywords: regression)


(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:

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
NI:myself to request uplift to Aurora-46.
Flags: needinfo?(gsquelart)
Closed: 7 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:
[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]

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]

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]

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.
Flags: needinfo?(gsquelart)
You need to log in before you can comment on or make changes to this bug.