Re: mpv: add youtube-dl to RUN_DEPENDS

2022-02-27 Thread Gleydson Soares
>yt-dlp would probably be a better choice (mpv prefers it over
>youtube-dl).

sure, i couldn't agree more about it.
diff updated, ok?

Index: Makefile
===
RCS file: /cvs/ports/multimedia/mpv/Makefile,v
retrieving revision 1.79
diff -u -p -u -p -r1.79 Makefile
--- Makefile29 Jan 2022 12:30:14 -  1.79
+++ Makefile27 Feb 2022 18:34:42 -
@@ -6,6 +6,8 @@ GH_ACCOUNT =mpv-player
 GH_PROJECT =   mpv
 GH_TAGNAME =   v0.34.1
 
+REVISION = 0
+
 SHARED_LIBS += mpv 0.2 # 1.109
 
 CATEGORIES =   multimedia x11
@@ -54,6 +56,7 @@ LIB_DEPENDS = archivers/libarchive \
 
 MODPY_RUNDEP = No
 RUN_DEPENDS =  devel/desktop-file-utils \
+   www/yt-dlp \
x11/gtk+3,-guic
 
 SEPARATE_BUILD =   Yes


Re: mpv: add youtube-dl to RUN_DEPENDS

2022-02-27 Thread Stuart Henderson

yt-dlp would probably be a better choice (mpv prefers it over youtube-dl).

--
 Sent from a phone, apologies for poor formatting.

On 27 February 2022 14:24:32 Gleydson Soares  wrote:


the following patch adds youtube-dl to mpv's RUN_DEPENDS.

$ mpv --no-video https://www.youtube.com/watch?v=...
[ytdl_hook]
[ytdl_hook] youtube-dl failed: not found or not enough permissions
Failed to recognize file format.

OK?

Thanks,
gsoares




mpv: add youtube-dl to RUN_DEPENDS

2022-02-27 Thread Gleydson Soares
the following patch adds youtube-dl to mpv's RUN_DEPENDS.

$ mpv --no-video https://www.youtube.com/watch?v=...
[ytdl_hook]
[ytdl_hook] youtube-dl failed: not found or not enough permissions
Failed to recognize file format.

OK?

Thanks,
gsoares
Index: Makefile
===
RCS file: /cvs/ports/multimedia/mpv/Makefile,v
retrieving revision 1.79
diff -u -p -u -p -r1.79 Makefile
--- Makefile29 Jan 2022 12:30:14 -  1.79
+++ Makefile27 Feb 2022 13:06:35 -
@@ -6,6 +6,8 @@ GH_ACCOUNT =mpv-player
 GH_PROJECT =   mpv
 GH_TAGNAME =   v0.34.1
 
+REVISION = 0
+
 SHARED_LIBS += mpv 0.2 # 1.109
 
 CATEGORIES =   multimedia x11
@@ -54,6 +56,7 @@ LIB_DEPENDS = archivers/libarchive \
 
 MODPY_RUNDEP = No
 RUN_DEPENDS =  devel/desktop-file-utils \
+   www/youtube-dl \
x11/gtk+3,-guic
 
 SEPARATE_BUILD =   Yes