search

mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
27.88k stars 2.87k forks source link

Replace youtube-dl with a fork #9093

Closed NoakPalander closed 3 years ago

NoakPalander commented 3 years ago

Expected behavior of the wanted feature

For it to be able to play age restricted videos by running e.g mpv https://youtube.com/etc/etc. Currently doesn't work with youtube-dl that mpv relies on, however there's a fork called yt-dlp on github that solves this.

sfan5 commented 3 years ago

You can already change the executable mpv calls to yt-dlp, there's nothing we have to do for this to work. duplicate of #8665

snylonue commented 3 years ago

try --ytdl_path though I don't know if it will work

garoto commented 3 years ago

It's a script-opt: --script-opts=ytdl_hook-ytdl_path=yt-dlp

NoakPalander commented 3 years ago

Was unaware that it was a duplicate, sorry for that. And also thank you for pointing out the script-opt flag.

mradalbert commented 3 years ago

youtube-dl maintainers went silent for 3 months and developement is non existent right now. Many users and devs migrated to yt-dlp already, so it would be preferable if mpv at least tried to use yt-dlp in absence of youtube-dl by default without need of modifying scripts by user, as future of youtube-dl it uncertain at this point.

mradalbert commented 3 years ago

https://github.com/ytdl-org/youtube-dl/issues/29753

mradalbert commented 3 years ago

Linux users can install mpv + yt-dlp using GUI package manager (yt-dlp is at least in Arch User Repository) and modifying scripts or installing additional wrapper worsens experience for new/inexperienced users.