[gentoo-commits] repo/gentoo:master commit in: net-misc/ytfzf/

["Ionen Wolkens" <ionen@gentoo.org>]

commit:     3e6b751c1cc24519d4c10e16987bbe87d8f16e7c
Author:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
AuthorDate: Fri Jul 15 14:49:03 2022 +0000
Commit:     Ionen Wolkens <ionen <AT> gentoo <DOT> org>
CommitDate: Fri Jul 15 15:11:12 2022 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=3e6b751c

net-misc/ytfzf: add IUSE=X for ueberzug, adjust postinst message

On second thought, may be fair to provide upstream's default, but given
ueberzug is X-only put it behind IUSE=X (also allows to auto-enable on
desktop profiles and such). No intention to add separate use for every
option, so not using IUSE=ueberzug.

IUSE=minimal will still disable everything optional should users want
to use this differently (e.g. I use kitty to display images rather than
ueberzug). Currently no straight-forward choice for wayland, so won't
provide a IUSE=wayland at the moment (could use chafa but well, not
high quality display and still requires changing settings).

Closes: https://bugs.gentoo.org/855179
Signed-off-by: Ionen Wolkens <ionen <AT> gentoo.org>

 net-misc/ytfzf/ytfzf-2.4.0.ebuild | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/net-misc/ytfzf/ytfzf-2.4.0.ebuild b/net-misc/ytfzf/ytfzf-2.4.0.ebuild
index aabfab731b1b..36a2a31e9d01 100644
--- a/net-misc/ytfzf/ytfzf-2.4.0.ebuild
+++ b/net-misc/ytfzf/ytfzf-2.4.0.ebuild
@@ -12,15 +12,14 @@ SRC_URI="https://github.com/pystardust/ytfzf/archive/refs/tags/v${PV}.tar.gz ->
 LICENSE="GPL-3"
 SLOT="0"
 KEYWORDS="~amd64 ~x86"
-IUSE="minimal"
+IUSE="X minimal"
 
-# fzf/mpv/yt-dlp "can" be optfeatures depending on configuration, but depend
-# on them so it works as expected out-of-the-box while allowing to disable.
 RDEPEND="
 	app-misc/jq
 	net-misc/curl[ssl]
 	virtual/awk
 	!minimal? (
+		X? ( media-gfx/ueberzug )
 		app-shells/fzf
 		media-video/mpv[lua]
 		net-misc/yt-dlp
@@ -49,12 +48,12 @@ src_install() {
 
 pkg_postinst() {
 	optfeature "external menu support" x11-misc/dmenu
-	optfeature "in-terminal thumbnails on X11" media-gfx/ueberzug
 	optfeature "desktop notifications" x11-libs/libnotify
 
 	if [[ ! ${REPLACING_VERSIONS} ]]; then
 		elog "Note that ${PN} supports many methods to display menus/thumbnails."
 		elog "This ebuild primarily covers defaults and major features, additional"
-		elog "dependencies may be needed for others."
+		elog "dependencies may be needed for others. Set USE=minimal if want full"
+		elog "control over optional dependencies (e.g. fzf is optional if use dmenu)."
 	fi
 }