Package Details: yt-dlp-git 2024.03.10.r9.g86d2f4d-1

Git Clone URL: https://aur.archlinux.org/yt-dlp-git.git (read-only, click to copy)
Package Base: yt-dlp-git
Description: A youtube-dl fork with additional features and fixes (git)
Upstream URL: https://github.com/yt-dlp/yt-dlp
Licenses: Unlicense
Conflicts: yt-dlp
Provides: yt-dlp
Submitter: katt
Maintainer: katt
Last Packager: katt
Votes: 43
Popularity: 0.60
First Submitted: 2021-08-08 15:20 (UTC)
Last Updated: 2024-03-17 17:40 (UTC)

Dependencies (24)

Required by (117)

Sources (1)

Latest Comments

« First ‹ Previous 1 2 3 4 5 6 7 8 9 Next › Last »

ruro commented on 2023-07-30 10:56 (UTC)

@katt just to be clear, the issue was already raised upstream and the official answer was that 1) they don't "recommend" installing the package without its "optional" dependencies and 2) if downstream distributors insist on providing a "dependency-free" install, they should patch out the missing dependencies from requirements.txt or install_requires in setup.py.

Currently, pip check CORRECTLY complains about the missing dependencies that are currently declared as non-optional in the installed python package metadata. Either these packages should be installed, or they should be removed from the python metadata. Your lack of care is somewhat understandable, but I think that "well, it works for me" isn't a very productive attitude to have for package maintainers.

katt commented on 2023-07-30 10:39 (UTC)

@ruro You should report this to the package in the repos if you really think it's a problem.

Also as to what pip check complains about I couldn't possibly care less about. The application works perfectly without them installed (aside for the optional parts, of course).

ruro commented on 2023-07-30 07:25 (UTC)

@katt yeah, but they aren't actually optional (despite what the readme says) according to the upstream maintainers. See https://github.com/yt-dlp/yt-dlp/issues/6484. For example, pip check currently complains if these dependencies are missing. They should either be marked as non-optional dependencies (like the upstream is suggesting) or you should patch setup.py/requirements.txt so that these dependencies aren't treated as strictly required.

katt commented on 2023-07-30 07:16 (UTC)

@ruro python-mutagen and python-pycryptodomex is already in optdepends.

ruro commented on 2023-07-29 22:47 (UTC)

This package should probably declare python-brotli, python-mutagen and python-pycryptodomex as dependencies, since the upstream package depends on them.

drws commented on 2023-05-17 03:24 (UTC)

Probably 'phantomjs' should be listed as an optional dependency.

katt commented on 2023-05-09 14:10 (UTC) (edited on 2023-05-09 14:11 (UTC) by katt)

I'm not really a fan of the messy solution rekman came up with, unsafe ls or the solution gesh posted, the latter since it just seems to pick the first item in dist, which seems the opposite of what we want.

Either way while writing this I went and looked on the wiki page and found there are new guidelines for python VCS packages, and I will implement them shortly.

https://wiki.archlinux.org/title/Python_package_guidelines#Standards_based_(PEP_517)

rekman commented on 2023-05-09 13:58 (UTC)

I'm still not 100% sold because we shouldn't really trust times. The system clock could change for any number of reasons, daylight savings, the time zone could change. Corner cases? Yes, but these are some of the worst to debug.

gesh commented on 2023-05-09 13:42 (UTC) (edited on 2023-05-09 13:42 (UTC) by gesh)

OK, then adapt the advice in FAQ099 wheels=(dist/*.whl); python -m installer … "${wheels[0]}"

« First ‹ Previous 1 2 3 4 5 6 7 8 9 Next › Last »