Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527726 - app-forensics/ovaldi should only use RDEPEND for SELinux policy package dependency
Summary: app-forensics/ovaldi should only use RDEPEND for SELinux policy package depen...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: LABBE Corentin
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527698
  Show dependency tree
 
Reported: 2014-11-01 09:51 UTC by Sven Vermeulen (RETIRED)
Modified: 2014-11-02 08:07 UTC (History)
2 users (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:51:57 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:52:15 UTC
Index: ovaldi-5.10.1.2.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-forensics/ovaldi/ovaldi-5.10.1.2.ebuild,v
retrieving revision 1.2
diff -u -B -r1.2 ovaldi-5.10.1.2.ebuild
--- ovaldi-5.10.1.2.ebuild      1 Mar 2014 22:12:04 -0000       1.2
+++ ovaldi-5.10.1.2.ebuild      1 Nov 2014 09:51:10 -0000
@@ -20,9 +20,9 @@
        dev-libs/libpcre
        dev-libs/xalan-c
        dev-libs/xerces-c
-       ldap? ( net-nds/openldap )
+       ldap? ( net-nds/openldap )"
+RDEPEND="${DEPEND}
        selinux? ( sys-libs/libselinux )"
-RDEPEND="${DEPEND}"
 
 S="${WORKDIR}/${P}-src"
 
Index: ovaldi-5.10.1.4.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/app-forensics/ovaldi/ovaldi-5.10.1.4.ebuild,v
retrieving revision 1.2
diff -u -B -r1.2 ovaldi-5.10.1.4.ebuild
--- ovaldi-5.10.1.4.ebuild      1 Mar 2014 22:12:04 -0000       1.2
+++ ovaldi-5.10.1.4.ebuild      1 Nov 2014 09:51:10 -0000
@@ -15,7 +15,7 @@
 KEYWORDS="~amd64 ~x86"
 IUSE="acl ldap rpm selinux"
 
-RDEPEND="dev-libs/libgcrypt:0
+CDEPEND="dev-libs/libgcrypt:0
        dev-libs/libpcre
        dev-libs/xalan-c
        dev-libs/xerces-c
@@ -23,11 +23,11 @@
        sys-libs/libcap
        acl? ( sys-apps/acl )
        ldap? ( net-nds/openldap )
-       rpm? ( app-arch/rpm )
-       selinux? ( sys-libs/libselinux )"
-DEPEND="${RDEPEND}
+       rpm? ( app-arch/rpm )"
+DEPEND="${CDEPEND}
        sys-apps/sed"
-
+RDEPEND="${CDEPEND}
+       selinux? ( sys-libs/libselinux )"
 S="${WORKDIR}/${P}-src"
 
 src_prepare() {
Comment 2 Sven Vermeulen (RETIRED) gentoo-dev 2014-11-02 08:07:20 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.