Bug 527820 – net-nds/rpcbind should only use RDEPEND for SELinux policy package dependency

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 11:57:33 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 11:57:44 UTC]

Index: rpcbind-0.2.1-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-nds/rpcbind/rpcbind-0.2.1-r1.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 rpcbind-0.2.1-r1.ebuild
--- rpcbind-0.2.1-r1.ebuild     30 Jan 2014 21:04:39 -0000      1.1
+++ rpcbind-0.2.1-r1.ebuild     1 Nov 2014 11:57:03 -0000
@@ -21,12 +21,13 @@
 SLOT="0"
 IUSE="debug selinux tcpd warmstarts"
 
-RDEPEND=">=net-libs/libtirpc-0.2.3
-       selinux? ( sec-policy/selinux-rpcbind )
+CDEPEND=">=net-libs/libtirpc-0.2.3
        tcpd? ( sys-apps/tcp-wrappers )"
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        virtual/pkgconfig"
-
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-rpcbind )
+"
 src_prepare() {
        [[ ${PV} == "9999" ]] && eautoreconf
        epatch_user
Index: rpcbind-9999.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-nds/rpcbind/rpcbind-9999.ebuild,v
retrieving revision 1.11
diff -u -B -r1.11 rpcbind-9999.ebuild
--- rpcbind-9999.ebuild 30 Jan 2014 21:06:30 -0000      1.11
+++ rpcbind-9999.ebuild 1 Nov 2014 11:57:03 -0000
@@ -21,12 +21,13 @@
 SLOT="0"
 IUSE="debug selinux tcpd warmstarts"
 
-RDEPEND=">=net-libs/libtirpc-0.2.3
-       selinux? ( sec-policy/selinux-rpcbind )
+CDEPEND=">=net-libs/libtirpc-0.2.3
        tcpd? ( sys-apps/tcp-wrappers )"
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        virtual/pkgconfig"
-
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-rpcbind )
+"
 src_prepare() {
        [[ ${PV} == "9999" ]] && eautoreconf
        epatch_user

[Comment 2 Sven Vermeulen (RETIRED) 2014-11-02 09:25:36 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.