FS#79340 - [yt-dlp] Missing 6 optional dependencies, one explicit too many

Attached to Project: Arch Linux
Opened by drws (drws) - Wednesday, 09 August 2023, 20:16 GMT
Last edited by Antonio Rojas (arojas) - Monday, 28 August 2023, 08:25 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jaroslav Lichtblau (Dragonlord)
Antonio Rojas (arojas)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: Going through README.md of the source repo yields additional optional dependencies missing from the current PKGBUILD. The exact package names from Arch repositories are listed below with extracted short descriptions.

- python-brotli: Brotli content encoding support
- python-brotlicffi: Brotli content encoding support
- python-xattr: For writing xattr metadata
- python-pyxattr: For writing xattr metadata (alternative option)
- phantomjs: For extractors with javascript code
- python-secretstorage: For '--cookies-from-browser' to access the GNOME keyring while decrypting cookies of Chromium-based browsers

The README also clearly states that python-certifi is an *optional* dependency while it is currently designated as an explicit one. This probably needs a correction.

Additional info:
* package version: 2023.07.06-1
This task depends upon

Closed by  Antonio Rojas (arojas)
Monday, 28 August 2023, 08:25 GMT
Reason for closing:  Fixed
Comment by drws (drws) - Monday, 28 August 2023, 04:02 GMT
The issue is about 6 missing dependencies and 1 possibly wrongly declared explicit. While the solution completely solves the missing dependencies it contains zero information about explicit python-certifi dependency. Source repo declares it optional. Is there a reason for explicit relation (or it possibly needs a correction)?
Comment by Toolybird (Toolybird) - Monday, 28 August 2023, 04:04 GMT
FWIW, it seems to successfully download a basic YT url without certifi. Refer also related discussion in  FS#79254 .
Comment by Antonio Rojas (arojas) - Monday, 28 August 2023, 08:25 GMT
certifi is a minimal wrapper over ca-certificates and may be required for some sites. I will leave it as a dependency.

Loading...