Bug 527798 – net-misc/chrony should only use RDEPEND for SELinux policy package dependency

• Product: Gentoo Linux
• Status: RESOLVED OBSOLETE

[Description Sven Vermeulen (RETIRED) 2014-11-01 11:34:16 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:34:25 UTC]

Index: chrony-1.30-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-misc/chrony/chrony-1.30-r1.ebuild,v
retrieving revision 1.1
diff -u -B -r1.1 chrony-1.30-r1.ebuild
--- chrony-1.30-r1.ebuild       6 Aug 2014 09:44:31 -0000       1.1
+++ chrony-1.30-r1.ebuild       1 Nov 2014 11:33:41 -0000
@@ -14,15 +14,17 @@
 KEYWORDS="amd64 ~arm hppa ~mips ppc sparc x86"
 IUSE="caps ipv6 +readline +rtc selinux"
 
-RDEPEND="
+CDEPEND="
        caps? ( sys-libs/libcap )
        readline? ( >=sys-libs/readline-4.1-r4 )
-       selinux? ( sec-policy/selinux-chronyd )
 "
 DEPEND="
-       ${RDEPEND}
+       ${CDEPEND}
        sys-apps/texinfo
 "
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-chronyd )
+"
 
 RESTRICT=test
 
Index: chrony-1.31.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-misc/chrony/chrony-1.31.ebuild,v
retrieving revision 1.6
diff -u -B -r1.6 chrony-1.31.ebuild
--- chrony-1.31.ebuild  30 Oct 2014 09:55:39 -0000      1.6
+++ chrony-1.31.ebuild  1 Nov 2014 11:33:41 -0000
@@ -14,15 +14,17 @@
 KEYWORDS="amd64 ~arm hppa ~mips ~ppc sparc x86"
 IUSE="caps ipv6 +readline +rtc selinux"
 
-RDEPEND="
+CDEPEND="
        caps? ( sys-libs/libcap )
        readline? ( >=sys-libs/readline-4.1-r4 )
-       selinux? ( sec-policy/selinux-chronyd )
 "
 DEPEND="
-       ${RDEPEND}
+       ${CDEPEND}
        sys-apps/texinfo
 "
+RDEPEND="${CDEPEND}
+       selinux? ( sec-policy/selinux-chronyd )
+"
 
 RESTRICT=test

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