Re: yt-dlp missing 3 dependencies?

On 2024/10/16 23:37, Stuart Henderson wrote:
> On 2024/10/15 19:15, Anon Loli wrote:
> > hello, folks
> >
> > I had previously compiled OpenBSD by myself and applied some azalia patch which
> > fixed my computer for OpenBSD.
> >
> > After doing sysupgrade the other day, to 7.6, yt-dlp seems to be broken.
> > I even tried deleting everything like so: pkg_delete -Xa, which did not help.
> >
> > The fix is installing the following 3 libraries:
> > py-certifi
> > py-urllib3
> > py-requests
>
> or more, from pyproject.yaml:
>
> dependencies = [
> "brotli; implementation_name=='cpython'",
> "brotlicffi; implementation_name!='cpython'",
> "certifi",
> "mutagen",
> "pycryptodomex",
> "requests>=2.32.2,<3",
> "urllib3>=1.26.17,<3",
> "websockets>=13.0",
> ]
>
> some are probably only used for certain site handlers but probably
> makes sense to follow upstream's list.

oops, sent too soon - wrong path for certifi and we don't have a
cryptodomex port (we probably should) but the main user for the latter
is hls which can use ffmpeg instead:

Index: Makefile
===================================================================
RCS file: /cvs/ports/www/yt-dlp/Makefile,v
diff -u -p -r1.43 Makefile
--- Makefile 29 Sep 2024 19:56:58 -0000 1.43
+++ Makefile 16 Oct 2024 22:57:32 -0000
@@ -2,6 +2,7 @@ COMMENT = CLI program to download videos

VERSION = 2024.09.27
MODPY_EGG_VERSION = ${VERSION:S/.0/./g}
+REVISION = 0

DISTNAME = yt-dlp-${VERSION}
DISTFILES = yt-dlp-${VERSION}{yt-dlp}${EXTRACT_SUFX}
@@ -18,6 +19,17 @@ SITES = https://github.com/yt-dlp/yt-dl
MODULES = lang/python

MODPY_PYBUILD = hatchling
+
+# from pyproject.toml; also lists cryptodomex which we don't have in
+# ports but the main use is for hls which can use ffnpeg instead
+RUN_DEPENDS = archivers/py-brotli${MODPY_FLAVOR} \
+ archivers/py-brotlicffi${MODPY_FLAVOR} \
+ devel/py-certifi${MODPY_FLAVOR} \
+ audio/py-mutagen${MODPY_FLAVOR} \
+ www/py-requests${MODPY_FLAVOR} \
+ www/py-urllib3${MODPY_FLAVOR} \
+ net/py-websockets${MODPY_FLAVOR}
+RUN_DEPENDS += graphics/ffmpeg

TEST_DEPENDS += devel/flake8