Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 550822 - apache-2.eclass : should RDEPEND on selinux? sec-policy/selinux-apache
Summary: apache-2.eclass : should RDEPEND on selinux? sec-policy/selinux-apache
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 527868
  Show dependency tree
 
Reported: 2015-05-30 14:00 UTC by Jason Zaman
Modified: 2015-05-30 14:12 UTC (History)
1 user (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Zaman gentoo-dev 2015-05-30 14:00:26 UTC 
SELinux deps should only be in RDEPEND not DEPEND. See the blocker bug for more details.
Something like this patch should work:

Index: apache-2.eclass
===================================================================
RCS file: /var/cvsroot/gentoo-x86/eclass/apache-2.eclass,v
retrieving revision 1.39
diff -u -B -r1.39 apache-2.eclass
--- apache-2.eclass	7 Aug 2014 07:37:54 -0000	1.39
+++ apache-2.eclass	30 May 2015 13:55:48 -0000
@@ -105,10 +105,10 @@
 	apache2_modules_deflate? ( sys-libs/zlib )
 	apache2_modules_mime? ( app-misc/mime-types )
 	ldap? ( =net-nds/openldap-2* )
-	selinux? ( sec-policy/selinux-apache )
 	ssl? ( >=dev-libs/openssl-0.9.8m )
 	!=www-servers/apache-1*"
-RDEPEND+=" ${DEPEND}"
+RDEPEND+=" ${DEPEND}
+	selinux? ( sec-policy/selinux-apache )"
 PDEPEND="~app-admin/apache-tools-${PV}"
 
 S="${WORKDIR}/httpd-${PV}"


also, the eclass maintainer email does not seem to exist in bugzie:
# @MAINTAINER:
# apache-devs@gentoo.org
Comment 1 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2015-05-30 14:12:01 UTC 
+  30 May 2015; Lars Wendler <polynomial-c@gentoo.org> apache-2.eclass:
+  Moved selinux dependency from DEPEND to RDEPEND (bug #550822). Fixed
+  maintainer email in eclass.
+