Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527716 - app-crypt/mit-krb5 should only use RDEPEND for SELinux policy package dependency
Summary: app-crypt/mit-krb5 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 Kerberos Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 09:31 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 07:58 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 09:31:15 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 09:31:33 UTC
Index: mit-krb5-1.12.2.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-crypt/mit-krb5/mit-krb5-1.12.2.ebuild,v
retrieving revision 1.10
diff -u -B -r1.10 mit-krb5-1.12.2.ebuild
--- mit-krb5-1.12.2.ebuild      19 Sep 2014 10:34:32 -0000      1.10
+++ mit-krb5-1.12.2.ebuild      1 Nov 2014 09:31:24 -0000
@@ -18,7 +18,7 @@
 KEYWORDS="alpha amd64 arm ~arm64 hppa ia64 ~mips ppc ppc64 ~s390 ~sh sparc x86"
 IUSE="doc +keyutils openldap +pkinit selinux +threads test xinetd"
 
-RDEPEND="!!app-crypt/heimdal
+CDEPEND="!!app-crypt/heimdal
        >=sys-libs/e2fsprogs-libs-1.42.9[${MULTILIB_USEDEP}]
        || ( >=dev-libs/libverto-0.2.5[libev,${MULTILIB_USEDEP}]
                >=dev-libs/libverto-0.2.5[libevent,${MULTILIB_USEDEP}]
@@ -26,19 +26,20 @@
        keyutils? ( >=sys-apps/keyutils-1.5.8[${MULTILIB_USEDEP}] )
        openldap? ( >=net-nds/openldap-2.4.38-r1[${MULTILIB_USEDEP}] )
        pkinit? ( >=dev-libs/openssl-1.0.1h-r2[${MULTILIB_USEDEP}] )
-       selinux? ( sec-policy/selinux-kerberos )
        xinetd? ( sys-apps/xinetd )
        abi_x86_32? (
                !<=app-emulation/emul-linux-x86-baselibs-20140508-r1
                !app-emulation/emul-linux-x86-baselibs[-abi_x86_32(-)]
        )"
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        ${PYTHON_DEPS}
        virtual/yacc
        doc? ( virtual/latex-base )
        test? ( ${PYTHON_DEPS}
                        dev-lang/tcl
                        dev-util/dejagnu )"
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-kerberos )"
 
 S=${WORKDIR}/${MY_P}/src
 
Index: mit-krb5-1.13.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-crypt/mit-krb5/mit-krb5-1.13.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 mit-krb5-1.13.ebuild
--- mit-krb5-1.13.ebuild        16 Oct 2014 09:12:47 -0000      1.1
+++ mit-krb5-1.13.ebuild        1 Nov 2014 09:31:24 -0000
@@ -17,7 +17,7 @@
 KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~hppa ~ia64 ~mips ~ppc ~ppc64 ~s390 ~sh ~sparc ~x86"
 IUSE="doc +keyutils openldap +pkinit selinux +threads test xinetd"
 
-RDEPEND="!!app-crypt/heimdal
+CDEPEND="!!app-crypt/heimdal
        >=sys-libs/e2fsprogs-libs-1.42.9[${MULTILIB_USEDEP}]
        || ( >=dev-libs/libverto-0.2.5[libev,${MULTILIB_USEDEP}]
                >=dev-libs/libverto-0.2.5[libevent,${MULTILIB_USEDEP}]
@@ -25,19 +25,20 @@
        keyutils? ( >=sys-apps/keyutils-1.5.8[${MULTILIB_USEDEP}] )
        openldap? ( >=net-nds/openldap-2.4.38-r1[${MULTILIB_USEDEP}] )
        pkinit? ( >=dev-libs/openssl-1.0.1h-r2[${MULTILIB_USEDEP}] )
-       selinux? ( sec-policy/selinux-kerberos )
        xinetd? ( sys-apps/xinetd )
        abi_x86_32? (
                !<=app-emulation/emul-linux-x86-baselibs-20140508-r1
                !app-emulation/emul-linux-x86-baselibs[-abi_x86_32(-)]
        )"
-DEPEND="${RDEPEND}
+DEPEND="${CDEPEND}
        ${PYTHON_DEPS}
        virtual/yacc
        doc? ( virtual/latex-base )
        test? ( ${PYTHON_DEPS}
                        dev-lang/tcl
                        dev-util/dejagnu )"
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-kerberos )"
 
 S=${WORKDIR}/${MY_P}/src
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 07:58:44 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.