Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 656360

Summary: sys-apps/firejail-0.9.52.ebuild: unnecessary dependency on x11-wm/xpra when using x11 USE flag
Product: Gentoo Linux Reporter: BetaRays
Component: Current packagesAssignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status: RESOLVED FIXED    
Severity: normal CC: jstein
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description BetaRays 2018-05-23 19:03:17 UTC
firejail does not strictly depend on xpra when using --enable-x11.
From the manual page
>       --x11  Sandbox the application using Xpra, Xephyr, Xvfb or Xorg security extension.  The sandbox will prevents screenshot and keylogger applications started inside the sandbox from accessing clients running outside the sandbox.   *Firejail  will try  first  Xpra, and if Xpra is not installed on the system, it will try to find Xephyr.*
It should not depend on xephyr either because the x11 USE flag allows using other options, including --x11=none (which shouldn't require any other package).
Comment 1 Larry the Git Cow gentoo-dev 2019-07-29 04:21:29 UTC
The bug has been referenced in the following commit(s):

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

commit ce1c4232290943a5132bd7f4cce426b8fc8521a0
Author:     Dennis Lamm <expeditioneer@gentoo.org>
AuthorDate: 2019-07-29 04:20:27 +0000
Commit:     Dennis Lamm <expeditioneer@gentoo.org>
CommitDate: 2019-07-29 04:21:12 +0000

    sys-apps/firejail: bump 0.9.60
    
    Bug: https://bugs.gentoo.org/656360
    Bug: https://bugs.gentoo.org/678976
    Signed-off-by: Dennis Lamm <expeditioneer@gentoo.org>
    Package-Manager: Portage-2.3.66, Repoman-2.3.16

 sys-apps/firejail/Manifest               |  1 +
 sys-apps/firejail/firejail-0.9.60.ebuild | 53 ++++++++++++++++++++++++++++++++
 sys-apps/firejail/metadata.xml           | 36 +++++++++++-----------
 3 files changed, 72 insertions(+), 18 deletions(-)
Comment 2 Dennis Lamm gentoo-dev 2019-07-30 19:13:57 UTC
fixed with 0.9.60 ebuild