Bug 527756 – mail-filter/opendkim should only use RDEPEND for SELinux policy package dependency

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 10:42:51 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 10:43:05 UTC]

Index: opendkim-2.9.2-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/mail-filter/opendkim/opendkim-2.9.2-r1.ebuild,v
retrieving revision 1.5
diff -u -B -r1.5 opendkim-2.9.2-r1.ebuild
--- opendkim-2.9.2-r1.ebuild    14 Aug 2014 16:17:22 -0000      1.5
+++ opendkim-2.9.2-r1.ebuild    1 Nov 2014 10:42:17 -0000
@@ -19,7 +19,7 @@
 KEYWORDS="amd64 ~arm x86"
 IUSE="+berkdb gnutls ldap lmdb lua memcached opendbx poll sasl selinux +ssl static-libs unbound"
 
-DEPEND="|| ( mail-filter/libmilter mail-mta/sendmail )
+CDEPEND="|| ( mail-filter/libmilter mail-mta/sendmail )
        dev-libs/libbsd
        ssl? ( >=dev-libs/openssl-0.9.8 )
        berkdb? ( >=sys-libs/db-3.2 )
@@ -29,12 +29,12 @@
        lmdb? ( dev-db/lmdb )
        memcached? ( dev-libs/libmemcached )
        sasl? ( dev-libs/cyrus-sasl )
-       selinux? ( sec-policy/selinux-dkim )
        unbound? ( >=net-dns/unbound-1.4.1 net-dns/dnssec-root )
        !unbound? ( net-libs/ldns )
        gnutls? ( >=net-libs/gnutls-2.11.7 )"
-
-RDEPEND="${DEPEND}
+DEPEND="${CDEPEND}"
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-dkim )
        sys-process/psmisc"
 
 REQUIRED_USE="sasl? ( ldap )"

[Comment 2 Sven Vermeulen (RETIRED) 2014-11-02 08:35:41 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.