Remove dependency on libavformat

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jya, Unassigned)

Tracking

43 Branch
mozilla44
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox43 fixed, firefox44 fixed)

Details

Attachments

(2 attachments)

Reporter

Description

4 years ago
When searching for ffmpeg or libav we currently look for libavformat and search a specific version.

We have no use for the libavformat library as we only use libavcodec and indirectly libavutil.

The version numbering in libavformat is also completely irrelevant for our use.

We should look for libavcodec instead.
Reporter

Comment 1

4 years ago
Interestingly, libavformat doesn't appear to be installed on ubuntu unless you install the full ffmpeg or gstreamer-ffmpeg plugin.

So currently to use Firefox with ffmpeg, you must install gstreamer-ffmpeg which is rather a pity
Reporter

Comment 2

4 years ago
Attachment #8667657 - Flags: review?(edwin)
Comment on attachment 8667657 [details] [diff] [review]
Remove dependency on libavformat.

Review of attachment 8667657 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/media/platforms/ffmpeg/FFmpegRuntimeLinker.cpp
@@ +30,5 @@
>  public:
>    static already_AddRefed<PlatformDecoderModule> Create();
>  };
>  
>  static const AvFormatLib sLibs[] = {

Rename |struct AvFormatLib|.
Attachment #8667657 - Flags: review?(edwin) → review+
Comment on attachment 8668286 [details] [diff] [review]
P2. Initialize PDM before attempting to use it.

Review of attachment 8668286 [details] [diff] [review]:
-----------------------------------------------------------------

(Jean-Yves requested I do the review, as we're not sure whether Edwin is available at the moment.)
Attachment #8668286 - Flags: review?(edwin) → review+
https://hg.mozilla.org/mozilla-central/rev/95fd37cd9c3e
https://hg.mozilla.org/mozilla-central/rev/9d7b74d45f71
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Reporter

Updated

4 years ago
Blocks: 1214943
You need to log in before you can comment on or make changes to this bug.