Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 454155 - munin-node should depend on more perl-modules
Summary: munin-node should depend on more perl-modules
Alias: None
Product: Fedora
Classification: Fedora
Component: munin
Version: 9
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-07-05 16:02 UTC by Roy-Magne Mo
Modified: 2009-01-14 05:37 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-01-14 05:37:25 UTC

Attachments (Terms of Use)

Description Roy-Magne Mo 2008-07-05 16:02:12 UTC
Description of problem:
perl dependencies in plugins are wrapped in eval to not make package managers
pick them up

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
Actual results:

Expected results:

Additional info:
munin-node really depends on a few perl modules to work right, these are wrapped
in evals to not fail while using automatic configuration. These modules should
be a dependency of munin-node. All these modules are either in the standard perl
distribution or in fedora and EPEL.

These modules should also be listed as a requirement:

# grep -wr eval /usr/share/munin/plugins/|grep -i require
/usr/share/munin/plugins/apache_accesses:if (! eval "require LWP::UserAgent;")
/usr/share/munin/plugins/apache_processes:if (! eval "require LWP::UserAgent;")
/usr/share/munin/plugins/apache_volume:if (! eval "require LWP::UserAgent;")
/usr/share/munin/plugins/dhcpd3:if(! eval "require Net::Netmask") {
/usr/share/munin/plugins/dhcpd3:if(! eval "require HTTP::Date") {
/usr/share/munin/plugins/ircu:if (! eval "require Net::IRC;")
/usr/share/munin/plugins/squid_cache:if (! eval "require IO::Socket;")
/usr/share/munin/plugins/squid_cache:if (! eval "require MIME::Base64;")
/usr/share/munin/plugins/squid_icp:if (! eval "require IO::Socket;")
/usr/share/munin/plugins/squid_icp:if (! eval "require MIME::Base64;")
/usr/share/munin/plugins/squid_icp:if (! eval "require Net::hostent;")
/usr/share/munin/plugins/squid_requests:if (! eval "require IO::Socket;")
/usr/share/munin/plugins/squid_requests:if (! eval "require MIME::Base64;")
/usr/share/munin/plugins/squid_traffic:if (! eval "require IO::Socket;")
/usr/share/munin/plugins/squid_traffic:if (! eval "require MIME::Base64;")

Comment 1 Kevin Fenzi 2008-07-12 16:11:07 UTC
I'm not sure about this one... upstream is making those conditional, ie it will
use them if they are installed, but won't cause an error if not. Should we
really require every munin-node install to have these optional modules even if
they don't run squid or apache or the like?

I guess another alternative would be to split out the plugins as subpackages,
but that could be a hassle to maintain, especially since they are all in the
same upstream source. 

Are you seeing problems due to this? 

Comment 2 Kevin Fenzi 2008-09-08 20:57:29 UTC
I'm inclined to just leave things as they are right now. 

Splitting out subpackages seems like a lot of overhead for single plugins, 
and requiring these things that only make sense for specific plugins would pull in tons of deps that 99% of people don't need. 

Perhaps adding a note about this to a README would suffice?

Comment 3 Kevin Fenzi 2009-01-14 05:17:06 UTC
I'm going to go ahead and close this now. 

Roy: If you have additional thoughts, or info here feel free to re-open...

Note You need to log in before you can comment on or make changes to this bug.