[Feature Request] Migrate Youtube-DL to yt-dlp

[seniorm0ment]

Suggesting moving from Youtube-DL to yt-dlp
https://github.com/yt-dlp/yt-dlp

yt-dlp is a fork of Youtube-DLC (https://github.com/blackjack4494/yt-dlc - inactive), which is a fork of Youtube-DL. It brings a bit of a faster feature development, as well as actively listening to the community, where youtube-dl has not been so amazing in that aspect. It comes with some new features as well.

[CounterPillow]

if the json dump invocation stayed the same don't need to move from anything to anything, it can just look for either of the two.

[Dudemanguy]

The youtube-dl hook is just a lua script. You can already set a custom path for whatever executable you want so there's no need for us to do anything. See the ytdl_path option in the manual.

[seniorm0ment]

Thank you for clarifying. Sorry for the bother.

[avahe-kellenberger]

Is this not available as an option in mpv.conf? I'd prefer to not have to pass a command line flag every time.

[JohnDDuncanIII]

@avahe-kellenberger

• ~/.config/mpv.conf

script-opts=ytdl_hook-ytdl_path="/opt/bin/yt-dlp"

then you can run a test with mpv -v <youtube_uri> to double check that it's using yt-dlp over youtube-dl

[ytdl_hook] full hook 
[ytdl_hook] found youtube-dl at: /opt/bin/yt-dlp 
[ytdl_hook] youtube-dl succeeded! 
[ytdl_hook] format selection: youtube-dl (separate) 

#8264 (comment)

[avih]

script-opts=ytdl_hook-ytdl_path="/opt/bin/yt-dlp"

You probably want this instead:

script-opts-add=ytdl_hook-ytdl_path="/opt/bin/yt-dlp"

Note the additional -add, because if you don't use it, it will wipe any prior script-opts which were set at mpv.conf, but with script-opts-add=... it will add this value instead of wiping all prior values.

[avahe-kellenberger]

Thanks!

[sebma]

The youtube-dl hook is just a lua script. You can already set a custom path for whatever executable you want so there's no need for us to do anything. See the ytdl_path option in the manual.

@Dudemanguy Strange, I see no ytdl_path option in the manual of mpv v0.32 :

$ man mpv | grep ytdl_path
$

Was this option introduced later ?

[Dudemanguy]

Was this option introduced later ?

Yes, 0.32 is too old.

[sebma]

@Dudemanguy

Was this option introduced later ?

Yes, 0.32 is too old.

It seems the ytdl_path of the ytdl_hook was introduced with mpv v0.33.0 :

$ git log v0.32.0..v0.33.0 | grep ytdl_path
    ytdl_hook: if ytdl not found in config dirs, use ytdl_path as is