Bug 6215

Summary: [PATCH] Add `Supplements: yt-dlp` to ffmpeg.
Product: Fedora Reporter: Maxwell G <gotmax>
Component: ffmpegAssignee: Dominik 'Rathann' Mierzejewski <dominik>
Status: RESOLVED INVALID    
Severity: enhancement CC: belegdol, kwizart, leigh123linux
Priority: P1    
Version: unspecified   
Hardware: x86_64   
OS: GNU/Linux   
namespace:
Attachments: [PATCH] Add `Supplements: yt-dlp` to ffmpeg.

Description Maxwell G 2022-02-11 23:07:18 CET 
Created attachment 2365 [details]
[PATCH] Add `Supplements: yt-dlp` to ffmpeg.

commit 20f3fc028525bd6cf14b7325798e6b920d79c167 (HEAD -> master)
Author: Maxwell G <gotmax@e.email>
Date:   Fri Feb 11 16:10:48 2022 -0600

    Add `Supplements: yt-dlp` to ffmpeg.
    
    yt-dlp does not have a hard dependency on ffmpeg, but upstream
    highly recommends installing it, as it is required for certain
    functions. A reverse weak dependency is required, as Fedora packages
    cannot (weakly) depend on RPMFusion packages.
    
    Signed-off-by: Maxwell G <gotmax@e.email>
Comment 1 Nicolas Chauvet 2022-02-13 10:14:21 CET 
I understand that it's how the "fedora project" wants to turn us into "slavery" by solving their packaging problem, and maintaining a tremendous list of packages that can be supplemented by ffmpeg, but we (ffmpeg active maintainers) do not consent to that.

The concern is that it create an incentive for every projects optionality using ffmpeg binaries to submit a similar bug. We we well end in a many to 1 ratio instead of having 1 to 1 ratio where maintainer just add Recommends: /usr/bin/ffmpeg even for the fedora cases.
Comment 2 Nicolas Chauvet 2022-02-13 10:22:32 CET 
Instead I recommend to use pkcon in your application to install the appropriate binaries at runtime. See PackageKit.

Also beware that fedora now provide a particularly disabled ffmpeg, so you will have to test codec availability if any users end with such package.
Comment 3 leigh scott 2022-02-14 07:20:40 CET 
File it against fedora ffmpeg.

https://koji.fedoraproject.org/koji/packageinfo?packageID=35247