Bug 259944 - multimedia/mpv - provide www/yt-dlp as default runtime dependency
Summary: multimedia/mpv - provide www/yt-dlp as default runtime dependency
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-20 09:05 UTC by Oleh Vinichenko
Modified: 2021-12-06 15:06 UTC (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oleh Vinichenko 2021-11-20 09:05:32 UTC 
www/youtube_dl is almost stopped in development, no tagged releases since June, 2021
Since mpv-0.34, if yt-dlp installed it will be used used by default at runtime.
Please, consider having yt-dlp as runtime dependency instead of youtube_dl, which is unconditional dependency in port, as well as binary package pulls in youtube_dl due to this. yt-dlp is an actively maintained fork of youtube_dl.
Comment 1 Joseph Mingrone freebsd_committer freebsd_triage 2021-11-23 03:30:00 UTC 
Review created: https://reviews.freebsd.org/D33090
Comment 2 commit-hook freebsd_committer freebsd_triage 2021-12-06 14:57:16 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=80e4e8275e9dfacaea51df8008738c9df3fd3522

commit 80e4e8275e9dfacaea51df8008738c9df3fd3522
Author:     Joseph Mingrone <jrm@FreeBSD.org>
AuthorDate: 2021-11-23 03:26:33 +0000
Commit:     Joseph Mingrone <jrm@FreeBSD.org>
CommitDate: 2021-12-06 14:09:50 +0000

    Ports that depend on youtube-dl: Depend on yt-dlp

    yt-dlp [1] is a youtube-dl fork with additional features and fixes.  For
    ports that depend on www/youtube-dl switch the dependency to www/yt-dlp.
    For ports that have an optional dependency on www/youtube-dl, create a
    new OPTIONS_RADIO that includes www/yt-dlp and make it the default.

    [1] https://github.com/yt-dlp/yt-dlp

    Other changes:
    - www/yt-dlp: Turn the symlink option on by default
    - www/py-pafy: Fix build when depending on www/yt-dlp

    PR:             259944
    Reported by:    Oleh Vinichenko <moonlapse81@gmail.com>
    Reviewed by:    diizzy, Oleh Vinichenko <moonlapse81@gmail.com>,
                    VVD <vvd@unislabs.com>
    Approved by:    0mp, adridg (kde@), rm, tcberner (multimedia@),
                    yuri (maintainers)
                    acm and nivit (maintainers, timeout)
    Differential Revision:  https://reviews.freebsd.org/D33090

 multimedia/gpodder/Makefile                          |  3 ++-
 multimedia/haruna/Makefile                           |  3 ++-
 multimedia/lives/Makefile                            |  8 +++++---
 multimedia/mpv/Makefile                              | 11 ++++++++---
 multimedia/smtube/Makefile                           |  3 ++-
 multimedia/ytfzf/Makefile                            |  3 ++-
 net-p2p/awgg/Makefile                                |  4 ++--
 www/py-pafy/Makefile                                 |  6 +++---
 www/py-pafy/files/patch-pafy_backend__youtube__dl.py |  8 +++++---
 www/py-pafy/files/patch-pafy_pafy.py                 |  8 +++++---
 www/yt-dlp/Makefile                                  |  2 +-
 11 files changed, 37 insertions(+), 22 deletions(-)
Comment 3 Joseph Mingrone freebsd_committer freebsd_triage 2021-12-06 15:06:35 UTC 
Thanks for reporting.  Committed.