Created attachment 246417 [details] Unfortunately py-SecretStorage pulls in a huge dependency list including rust but actually it's optional to yt-dlp. Please see https://github.com/yt-dlp/yt-dlp/pull/8585. Lets add an OPTION for that.
Unfortunately py-SecretStorage pulls in a huge dependency list including rust but actually it's optional to yt-dlp. Please see https://github.com/yt-dlp/yt-dlp/pull/8585. Lets add an OPTION for that.
The option description should help the user decide if they want that option, so it should tell the user what having that optional dependency provides, not what is done on the port level. Also the upstream default seems to be to depend on it, so it probably should be a port default, additionally having it as default avoids behaviour changes with the defaults.
(In reply to Benjamin Takacs from comment #2) Be my guest and modify.
A modified patch was committed. Thanks for the patch!
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=57a397e0a279cdb6e73a29d23e233051441d179c commit 57a397e0a279cdb6e73a29d23e233051441d179c Author: Yuri Victorovich <yuri@FreeBSD.org> AuthorDate: 2023-11-19 18:31:25 +0000 Commit: Yuri Victorovich <yuri@FreeBSD.org> CommitDate: 2023-11-19 18:33:55 +0000 www/yt-dlp: Make py-SecretStorage an optional port dependency py-SecretStorage pulls a lot of dependencies for a small optional feature in yt-dlp. The upstream merged the PR that also makes SecretStorage optional: https://github.com/yt-dlp/yt-dlp/pull/8585 PR: 275196 Submitted by: Helge Oldach <freebsd@oldach.net> (initial version) www/yt-dlp/Makefile | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-)