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
 }

Reply via email to