After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 797198 - Split out ffmpeg from gst-libav-1.0 to its own recipe
Split out ffmpeg from gst-libav-1.0 to its own recipe
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: cerbero
unspecified
Other All
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-09-24 15:14 UTC by Nirbheek Chauhan
Modified: 2018-09-26 12:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Split out ffmpeg from gst-libav-1.0 to its own recipe (13.08 KB, patch)
2018-09-24 15:14 UTC, Nirbheek Chauhan
none Details | Review
Split out ffmpeg from gst-libav-1.0 to its own recipe (14.06 KB, patch)
2018-09-26 05:19 UTC, Nirbheek Chauhan
committed Details | Review

Description Nirbheek Chauhan 2018-09-24 15:14:31 UTC
I haven't attached the Meson port of gst-libav here because I'm seeing some strange build failures in it.
Comment 1 Nirbheek Chauhan 2018-09-24 15:14:39 UTC
Created attachment 373747 [details] [review]
Split out ffmpeg from gst-libav-1.0 to its own recipe

This allows people to more easily replace the recipe with their own
ffmpeg, or change the configuration. Also allows us to move
gst-libav-1.0 to Meson.
Comment 2 Tim-Philipp Müller 2018-09-24 15:17:42 UTC
It's also needed for when we drop autotools in gst-libav (eventually).
Comment 3 Nicolas Dufresne (ndufresne) 2018-09-24 18:03:14 UTC
Review of attachment 373747 [details] [review]:

Maybe I'm confused, but iirc the LA generator was moved inside custom Makefile in gst-libav repo. We need to bring it back into cerbero now I guess.
Comment 4 Nirbheek Chauhan 2018-09-26 05:19:04 UTC
Created attachment 373766 [details] [review]
Split out ffmpeg from gst-libav-1.0 to its own recipe

This allows people to more easily replace the recipe with their own
ffmpeg, or change the configuration. Also allows us to move
gst-libav-1.0 to Meson.
Comment 5 Nirbheek Chauhan 2018-09-26 05:20:06 UTC
(In reply to Nicolas Dufresne (ndufresne) from comment #3)
> Review of attachment 373747 [details] [review] [review]:
> 
> Maybe I'm confused, but iirc the LA generator was moved inside custom
> Makefile in gst-libav repo. We need to bring it back into cerbero now I
> guess.

Fixed. Interestingly, the la files that were being generated contained `-llzma` which cerbero didn't provide, so they were wrong. I wonder why no one complained.
Comment 6 Nirbheek Chauhan 2018-09-26 05:29:29 UTC
Attachment 373766 [details] pushed as 3121278 - Split out ffmpeg from gst-libav-1.0 to its own recipe
Comment 7 Nicolas Dufresne (ndufresne) 2018-09-26 12:19:55 UTC
Yeah, the BZ/LZMA and many other things are particularly racy. We basically run an check in configure.ac, and hope configure in ffmpeg will get the same answer. But in theory, autotools buidls for Android should not have the ZMA lib unless it's found on the root fs.

Thanks for your work !