Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527706 - app-admin/logrotate should only use RDEPEND for SELinux policy package dependency
Summary: app-admin/logrotate should only use RDEPEND for SELinux policy package depend...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Chema Alonso Josa (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 08:59 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 07:47 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 08:59:25 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:02:57 UTC
Index: logrotate-3.8.7.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-admin/logrotate/logrotate-3.8.7.ebuild,v
retrieving revision 1.13
diff -u -B -r1.13 logrotate-3.8.7.ebuild
--- logrotate-3.8.7.ebuild      10 Jun 2014 00:06:56 -0000      1.13
+++ logrotate-3.8.7.ebuild      1 Nov 2014 09:02:32 -0000
@@ -15,15 +15,17 @@
 KEYWORDS="alpha amd64 arm arm64 hppa ia64 m68k ~mips ppc ppc64 s390 sh sparc x86 ~amd64-fbsd ~x86-fbsd"
 IUSE="acl selinux"
 
-RDEPEND="
+COMMON_DEPS="
        >=dev-libs/popt-1.5
        selinux? (
                sys-libs/libselinux
-               sec-policy/selinux-logrotate
        )
        acl? ( virtual/acl )"
 
-DEPEND="${RDEPEND}
+RDEPEND="${COMMON_DEPS}
+               selinux? ( sec-policy/selinux-logrotate )"
+
+DEPEND="${COMMON_DEPS}
        >=sys-apps/sed-4"
 
 src_prepare() {
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 07:47:24 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.