Bug 527908

Summary:	net-analyzer/munin lacks the selinux use flag
Product:	Gentoo Linux	Reporter:	Eric Gisse <jowr.pi>
Component:	SELinux	Assignee:	Sven Vermeulen (RETIRED) <swift>
Status:	RESOLVED FIXED
Severity:	normal	CC:	selinux, sysadmin
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	527868
Attachments:	munin ebuild patch

Description Eric Gisse 2014-11-01 20:33:14 UTC

Let's make selinux less horrible to use, if that's at all possible.

net-analyzer/munin won't function in an selinux environment by default due to not having a selinux use flag that pulls in the appropriate dependency ( sec-policy/selinux-munin ).


Reproducible: Always

Comment 1 Eric Gisse 2014-11-01 20:33:48 UTC

Created attachment 388282 [details, diff]
munin ebuild patch

This fixes the issue.

Comment 2 Sven Vermeulen (RETIRED) gentoo-dev

2014-11-02 14:21:26 UTC

Hi @sysadmin herd, would it be possible to add 

IUSE="selinux"
RDEPEND="selinux? ( sec-policy/selinux-munin )"

to the munin ebuild? I can do it for you (with revbump) if you don't have anything in the queue to be committed. Otherwise this can wait a bit if you believe that a new version/bump is going to be done soonish.

Index: munin-2.0.19-r1.ebuild
===================================================================
RCS file: /var/cvsroot/gentoo-x86/net-analyzer/munin/munin-2.0.19-r1.ebuild,v
retrieving revision 1.5
diff -u -B -r1.5 munin-2.0.19-r1.ebuild
--- munin-2.0.19-r1.ebuild      3 Aug 2014 18:32:34 -0000       1.5
+++ munin-2.0.19-r1.ebuild      2 Nov 2014 14:18:38 -0000
@@ -18,7 +18,7 @@
 LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="amd64 ~arm ~mips ppc x86"
-IUSE="asterisk irc java memcached minimal mysql postgres ssl test cgi ipv6 syslog ipmi http dhcpd doc apache"
+IUSE="asterisk irc java memcached minimal mysql postgres selinux ssl test cgi ipv6 syslog ipmi http dhcpd doc apache"
 REQUIRED_USE="cgi? ( !minimal ) apache? ( cgi )"
 
 # Upstream's listing of required modules is NOT correct!
@@ -94,6 +94,7 @@
                        virtual/cron
                        media-fonts/dejavu
                )
+               selinux? ( sec-policy/selinux-munin )
                !<sys-apps/openrc-0.11.8"
 
 S="${WORKDIR}/${MY_P}"

Comment 3 Sven Vermeulen (RETIRED) gentoo-dev

2015-01-03 12:14:21 UTC

Added to 2.0.25-r1 and 2.1.9999. I left 2.0.19 alone though, given that 2.0.25 is more likely to be stabilized anyway at the same time (or sooner) than a 2.0.19-r1 bump would.