New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update spec file #1283

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
5 participants
@xrobau
Copy link

xrobau commented Jul 31, 2014

This is a complete rewrite of the spec file. I've removed a lot of the legacy AIX code, as it'll never be used. I've also removed a lot of the install requirements, because people don't need to have - eg, - postgresql libraries installed if all they want to check is mysql.

@xrobau

This comment has been minimized.

Copy link

xrobau commented Jul 31, 2014

Resolves issue #1273

@skottler

This comment has been minimized.

Copy link
Member

skottler commented Jul 31, 2014

Thanks for the PR! I'll take a look at it tonight.

Name: monitoring-plugins
Version: @PACKAGE_VERSION@
Release: @RELEASE@
Summary: Host/service/network monitoring program plugins for Nagios and compatible

Summary: Host/service/network monitoring program plugins for Sensu/Icinga/Nagios etc..

This comment has been minimized.

@sni

sni Jul 31, 2014

Contributor

maybe we should mention shinken and naemon too or stay with "nagios compatible"

This comment has been minimized.

@waja

waja Jul 31, 2014

Member

I was aware of this and would have fixed that later with a follow-up commit! :-)

@xrobau

This comment has been minimized.

Copy link

xrobau commented Jul 31, 2014

Changed the summary to

Summary: Collection of monitoring plugins for Sensu/Icinga/Nagios and other Nagios compatible programs

How's that?

@waja

This comment has been minimized.

Copy link
Member

waja commented Aug 2, 2014

Any updates on this? Seems users are demanding rpm packages and all those enterprise distros seems still not to have monitoring-plugins available.

waja added a commit to waja/monitoring-plugins that referenced this pull request Oct 15, 2014

waja added a commit to waja/monitoring-plugins that referenced this pull request Oct 15, 2014

@waja waja added the enhancement label Oct 15, 2014

waja added a commit to waja/monitoring-plugins that referenced this pull request Oct 15, 2014

@waja waja force-pushed the monitoring-plugins:master branch 2 times, most recently from 441913d to 40c870e Oct 15, 2014

@waja waja added this to the 2.2 milestone Apr 13, 2015


%define npdir %{_builddir}/%{name}-%{version}
Packager: Rob Thomas <xrobau@gmail.com>

This comment has been minimized.

@glensc

glensc May 12, 2015

Contributor

you should not fill Packager: field, you won't be the one producing final .rpm


# Don't let rpmbuild try to figure out the dependancies.
# It's far too confusing for it.
AutoReqProv: no

This comment has been minimized.

@glensc

glensc May 12, 2015

Contributor

i'm sure this is very outdated nowadays.

the only reason for AutoReqProv: no is that you do not want to pull dependencies, but handle them yourself. this is not recommended, but then again allows installing the package so that postgresql packages are not pulled (and pgsql plugin will fail to find libraries)

@waja waja modified the milestones: 2.2, 2.3 Oct 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment