[Bug 1059281] Review Request: monitoring-plugins - Host/service/network monitoring program plugins for a variety of different systems

bugzilla at redhat.com bugzilla at redhat.com
Wed Jan 29 22:28:51 UTC 2014


https://bugzilla.redhat.com/show_bug.cgi?id=1059281



--- Comment #1 from Michael Scherer <misc at zarb.org> ---
So :
- the list on Requires on 1 single line is a bit too long for my taste, 1
requires per line would likely be better in diff, etc. And it would also be
better to have precise requires ( ie, on the specific version )

- the requires on the -log subpackage :
Requires: /bin/egrep

any reason to not use %{_bindir}/ like the rest ?

- description could be enhanced to say this provides more than plugin for
nagios 

- %{_libdir}/monitoring/ is unowned

- the various setuid plugin are not a good ideas IMHO, not sure if that
requires FESCO approval or something. I would feel better if this was using
capacity.

- I see a few tests for the plugins and the lib, yet no %check, is this normal
?

- there is a few tarballs of perl source code, but this is not used for now,
just mentioning for documenting it

- why do all plugin provides monitoring-plugins ? ( and not a requires ? )

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component


More information about the package-review mailing list