Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527816 - net-nds/openldap should only use RDEPEND for SELinux policy package dependency
Summary: net-nds/openldap should only use RDEPEND for SELinux policy package dependency
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo LDAP project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 11:53 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 09:20 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Vermeulen (RETIRED) gentoo-dev 2014-11-01 11:53:38 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) gentoo-dev 2014-11-01 11:53:51 UTC
Index: openldap-2.4.40-r2.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-nds/openldap/openldap-2.4.40-r2.ebuild,v
retrieving revision 1.3
diff -u -B -r1.3 openldap-2.4.40-r2.ebuild
--- openldap-2.4.40-r2.ebuild   1 Nov 2014 00:29:02 -0000       1.3
+++ openldap-2.4.40-r2.ebuild   1 Nov 2014 11:53:06 -0000
@@ -39,7 +39,7 @@
 for _slot in $BDB_SLOTS; do BDB_PKGS="${BDB_PKGS} sys-libs/db:${_slot}" ; done
 
 # openssl is needed to generate lanman-passwords required by samba
-RDEPEND="icu? ( dev-libs/icu:= )
+CDEPEND="icu? ( dev-libs/icu:= )
        ssl? ( !gnutls? ( >=dev-libs/openssl-1.0.1h-r2[${MULTILIB_USEDEP}] )
                gnutls? ( >=net-libs/gnutls-2.12.23-r6[${MULTILIB_USEDEP}] >=dev-libs/libgcrypt-1.5.3:0[${MULTILIB_USEDEP}] ) )
        sasl? ( dev-libs/cyrus-sasl:= )
@@ -64,14 +64,15 @@
                kerberos? ( virtual/krb5 )
                cxx? ( dev-libs/cyrus-sasl:= )
        )
-       selinux? ( sec-policy/selinux-ldap )
        abi_x86_32? (
                !<=app-emulation/emul-linux-x86-baselibs-20140508-r3
                !app-emulation/emul-linux-x86-baselibs[-abi_x86_32(-)]
        )"
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        sys-apps/groff"
-
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-ldap )
+"
 # for tracking versions
 OPENLDAP_VERSIONTAG=".version-tag"
 OPENLDAP_DEFAULTDIR_VERSIONTAG="/var/lib/openldap-data"
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 09:20:14 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.