Bug 527732 – app-misc/tmux should only use RDEPEND for SELinux policy package dependency

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 09:58:37 UTC]

Please remove the "selinux? ( sec-policy/selinux-* )" dependency in the DEPEND variable (not RDEPEND). Take this into account with your next package bump - if after 6 months there have been no need for other package upgrades, we will be bumping the package(s) ourselves with just this change.

For more information, see our tracker Bug #527698 [1]. The requested change will be posted as a separate comment/patch. Note that if the package also depends on libselinux, libselinux itself should still remain in DEPEND and RDEPEND.

[1] https://bugs.gentoo.org/show_bug.cgi?id=527698


Reproducible: Always

[Comment 1 Sven Vermeulen (RETIRED) 2014-11-01 09:58:55 UTC]

Index: tmux-1.9a.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-misc/tmux/tmux-1.9a.ebuild,v
retrieving revision 1.10
diff -u -B -r1.10 tmux-1.9a.ebuild
--- tmux-1.9a.ebuild    11 Oct 2014 14:00:58 -0000      1.10
+++ tmux-1.9a.ebuild    1 Nov 2014 09:58:46 -0000
@@ -18,14 +18,14 @@
 
 COMMON_DEPEND="
        >=dev-libs/libevent-2.0.10
-       sys-libs/ncurses
-       selinux? ( sec-policy/selinux-screen )"
+       sys-libs/ncurses"
 DEPEND="${COMMON_DEPEND}
        virtual/pkgconfig"
 RDEPEND="${COMMON_DEPEND}
        vim-syntax? ( || (
                app-editors/vim
-               app-editors/gvim ) )"
+               app-editors/gvim ) )
+       selinux? ( sec-policy/selinux-screen )"
 
 DOCS=( CHANGES FAQ README TODO )
 
Index: tmux-9999.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-misc/tmux/tmux-9999.ebuild,v
retrieving revision 1.12
diff -u -B -r1.12 tmux-9999.ebuild
--- tmux-9999.ebuild    17 May 2014 14:54:47 -0000      1.12
+++ tmux-9999.ebuild    1 Nov 2014 09:58:46 -0000
@@ -19,14 +19,14 @@
 
 COMMON_DEPEND="
        >=dev-libs/libevent-2.0.10
-       sys-libs/ncurses
-       selinux? ( sec-policy/selinux-screen )"
+       sys-libs/ncurses"
 DEPEND="${COMMON_DEPEND}
        virtual/pkgconfig"
 RDEPEND="${COMMON_DEPEND}
        vim-syntax? ( || (
                app-editors/vim
-               app-editors/gvim ) )"
+               app-editors/gvim ) )
+       selinux? ( sec-policy/selinux-screen )"
 
 DOCS=( CHANGES FAQ README TODO )

[Comment 2 Sven Vermeulen (RETIRED) 2014-11-02 08:09:20 UTC]

After checking with a few colleague developers it seems that this change does not need a revision bump after all. As such, I will make the necessary changes to the packages myself.

My apologies for the noise.