Bug 527772 – net-dns/unbound should only use RDEPEND for SELinux policy package dependency

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 11:00:27 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:00:35 UTC]

Index: unbound-1.4.22-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-dns/unbound/unbound-1.4.22-r1.ebuild,v
retrieving revision 1.2
diff -u -B -r1.2 unbound-1.4.22-r1.ebuild
--- unbound-1.4.22-r1.ebuild    23 Oct 2014 17:05:43 -0000      1.2
+++ unbound-1.4.22-r1.ebuild    1 Nov 2014 10:59:55 -0000
@@ -22,13 +22,15 @@
 # the executables. MULTILIB_USEDEP may be dropped once build system
 # is fixed.
 
-RDEPEND=">=dev-libs/expat-2.1.0-r3[${MULTILIB_USEDEP}]
+CDEPEND=">=dev-libs/expat-2.1.0-r3[${MULTILIB_USEDEP}]
        >=dev-libs/libevent-2.0.21[${MULTILIB_USEDEP}]
        >=dev-libs/openssl-1.0.1h-r2[${MULTILIB_USEDEP}]
        python? ( ${PYTHON_DEPS} )
-       selinux? ( sec-policy/selinux-bind )"
-
-DEPEND="${RDEPEND}
+"
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-bind )
+"
+DEPEND="${CDEPEND}
        python? ( dev-lang/swig )
        test? (
                net-dns/ldns-utils[examples]

[Comment 2 Tim Harder 2014-11-01 16:06:08 UTC]

(In reply to Sven Vermeulen from comment #0)
> 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.

Feel free to do this yourself in a revbump at anytime.

[Comment 3 Sven Vermeulen (RETIRED) 2014-11-02 08:48:42 UTC]

Thanks for the feedback,

After checking with a few colleague developers it seems that this change does not need a revision bump after all.

My apologies for the noise.