Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 736972 - x11-terms/terminator should conditionally depend on dev-libs/keybinder
Summary: x11-terms/terminator should conditionally depend on dev-libs/keybinder
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Alexey
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2020-08-13 11:51 UTC by Yaroslav Isakov
Modified: 2020-10-15 10:46 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yaroslav Isakov 2020-08-13 11:51:49 UTC
Right now terminator depends on keybinder, but it is needed only if display_manager() == 'X11'. Even if display manager is X11, and keybinder cannot be loaded, "hide_window shortcut will be unavailable". So, I think that it would be best to make this dep optional. Otherwise, it will introduce X11 library in Wayland-only system.
Comment 1 Larry the Git Cow gentoo-dev 2020-08-31 02:47:06 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4e3656e991694d5c401112513da593f519d79968

commit 4e3656e991694d5c401112513da593f519d79968
Author:     Alexey Sokolov <sokolov@google.com>
AuthorDate: 2020-08-13 22:45:19 +0000
Commit:     Stefan Strogin <steils@gentoo.org>
CommitDate: 2020-08-31 02:42:18 +0000

    x11-terms/terminator: move keybinder behind X USE flag
    
    Closes: https://bugs.gentoo.org/736972
    Package-Manager: Portage-2.3.103, Repoman-2.3.23
    Signed-off-by: Alexey Sokolov <alexey+gentoo@asokolov.org>
    Closes: https://github.com/gentoo/gentoo/pull/17117
    Signed-off-by: Stefan Strogin <steils@gentoo.org>

 x11-terms/terminator/terminator-1.92-r2.ebuild | 60 ++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)
Comment 2 Alexey 2020-10-15 10:46:58 UTC
FTR, keybinder is moving from X USE flag to optfeature: https://github.com/gentoo/gentoo/pull/17830