Bug 707692

Summary:	app-admin/keepassxc should not have any dependency on x11-misc/xsel
Product:	Gentoo Linux	Reporter:	Alex Xu (Hello71) <alex_y_xu>
Component:	Current packages	Assignee:	Lars Wendler (Polynomial-C) (RETIRED) <polynomial-c>
Status:	RESOLVED FIXED
Severity:	normal	CC:	arthur, fturco, yaroslav.isakov
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
See Also:	https://bugs.gentoo.org/show_bug.cgi?id=667092
Whiteboard:
Package list:		Runtime testing required:	---

Description Alex Xu (Hello71) 2020-01-31 23:18:19 UTC

keepassxc does not use and, to my knowledge, has never used xsel. I cloned the source and checked "git log -G xsel", with zero results. There are, however, several results for xclip. But even xclip is used only for clipboard access when using the command line interface, which is not the primary usage of keepassxc. Therefore, there is no reason to have it as a mandatory dependency. I also do not know why PDEPEND was used here, as there appears to be no circular dependency to resolve.

Comment 1 Alex Xu (Hello71) 2020-07-08 14:21:50 UTC

this is now stupid four times over instead of three, because not only is xclip used, not xsel, and there is no reason for PDEPEND, but additionally, according to 2.6.0 release notes, wl-clipboard is now used instead of xclip on wayland.

Comment 2 Larry the Git Cow gentoo-dev

2020-07-08 15:07:58 UTC

The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2fd8ba0d2ba198fe9708a9e61aca542d62bf3920

commit 2fd8ba0d2ba198fe9708a9e61aca542d62bf3920
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2020-07-08 15:06:58 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2020-07-08 15:07:52 +0000

    app-admin/keepassxc: Drop PDEPEND on x11-misc/xsel
    
    Closes: https://bugs.gentoo.org/707692
    Package-Manager: Portage-2.3.103, Repoman-2.3.23
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

 app-admin/keepassxc/keepassxc-2.6.0.ebuild | 4 ----
 1 file changed, 4 deletions(-)