On Thu, Sep 10, 2026 at 02:42:45PM +0100, Stuart Henderson wrote:
> On 2026/09/10 15:00, Robert Nilsson wrote:
> > Hello!
> > 
> > I think it would be great to have an option to install mpv
> > without the yt-dlp dependency.  Sometimes it's nice to just
> > have a good media player without all the stuff that comes
> > with yt-dlp.
> > 
> > Thanks.
> > 
> 
> With the addition of py-yt_dlp_ejs the deps for yt-dlp are indeed
> getting a bit heavy, so I think this would make sense. The error
> message when opening a URL if yt-dlp is not present looks like this:
> 
> [ytdl_hook] Subprocess failed: init
> [ytdl_hook] Subprocess failed: init
> [ytdl_hook] Subprocess failed: init
> [ytdl_hook]
> [ytdl_hook] youtube-dl failed: not found or not enough permissions
> Failed to recognize file format.
> 
> it would help if that message mentioned yt-dlp so that users wanting
> it have a better clue for which software to install, so here's a
> possible diff. Anyone have tweaks/objections?

Considering the youtube-dl homepage now redirects to yt-dlp, might be
best to only mention yt-dlp in the error message? either way ok lraab@

youtube-dl seems to be in some kind of no-man's-land. The homepage
redirects to yt-dlp, the repo though still has regular activity, but
none of the installation options barring "developer instructions"
would get you a reasonably up to date version which might work.

https://github.com/ytdl-org/youtube-dl#adding-support-for-a-new-site
https://yt-dl.org/

Thoughts on retiring youtube-dl?

> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/multimedia/mpv/Makefile,v
> diff -u -p -r1.110 Makefile
> --- Makefile	6 Mar 2026 09:00:41 -0000	1.110
> +++ Makefile	10 Sep 2026 13:40:50 -0000
> @@ -5,7 +5,7 @@ USE_NOBTCFI =		Yes
>  GH_ACCOUNT =		mpv-player
>  GH_PROJECT =		mpv
>  GH_TAGNAME =		v0.41.0
> -REVISION =		0
> +REVISION =		1
>  
>  SHARED_LIBS +=		mpv		2.1
>  
> @@ -53,7 +53,6 @@ LIB_DEPENDS =		archivers/libarchive \
>  MODPY_RUNDEP =		No
>  RUN_DEPENDS =		devel/desktop-file-utils \
>  			graphics/vulkan-loader \
> -			www/yt-dlp \
>  			x11/gtk+4,-guic
>  
>  CONFIGURE_ARGS =	-Daaudio=disabled \
> Index: patches/patch-player_lua_ytdl_hook_lua
> ===================================================================
> RCS file: patches/patch-player_lua_ytdl_hook_lua
> diff -N patches/patch-player_lua_ytdl_hook_lua
> --- /dev/null	1 Jan 1970 00:00:00 -0000
> +++ patches/patch-player_lua_ytdl_hook_lua	10 Sep 2026 13:40:50 -0000
> @@ -0,0 +1,12 @@
> +Index: player/lua/ytdl_hook.lua
> +--- player/lua/ytdl_hook.lua.orig
> ++++ player/lua/ytdl_hook.lua
> +@@ -986,7 +986,7 @@ local function run_ytdl_hook(url)
> +         -- trim our stderr to avoid spurious newlines
> +         local ytdl_err = result.stderr:gsub("^%s*(.-)%s*$", "%1")
> +         msg.error(ytdl_err)
> +-        local err = "youtube-dl failed: "
> ++        local err = "yt-dlp/youtube-dl failed: "
> +         if result.error_string and result.error_string == "init" then
> +             err = err .. "not found or not enough permissions"
> +         elseif parse_err then
>