Bug 202556

Summary: [PATCH] net-mgmt/nagios4: nagios_enable not set in rc script
Product: Ports & Packages Reporter: John Marshall <john>
Component: Individual Port(s)Assignee: Mathieu Arnold <mat>
Status: Closed FIXED    
Severity: Affects Some People Keywords: patch
Priority: --- Flags: bugzilla: maintainer-feedback? (mat)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Set default value "NO" for nagios_enable in rc script none

Description John Marshall 2015-08-21 21:32:44 UTC
Created attachment 160191 [details]
Set default value "NO" for nagios_enable in rc script

Comments at the beginning of the port's rc script say that nagios_enable is set to NO by default. This patch makes it so.

The rc script does not set a value for nagios_enable at all, so if the variable is not set in the system rc.conf, the rc system emits the following WARNING message during startup or shutdown if the port is installed but not configured:

  /usr/local/etc/rc.d/nagios: WARNING: $nagios_enable is not set properly - see rc.conf(5).

This patch modifies the port's rc script to set nagios_enable to "NO" if the variable is empty or not set in rc.conf.
Comment 1 commit-hook freebsd_committer freebsd_triage 2015-08-31 10:06:52 UTC
A commit references this bug:

Author: mat
Date: Mon Aug 31 10:06:08 UTC 2015
New revision: 395658
URL: https://svnweb.freebsd.org/changeset/ports/395658

Log:
  Set default value "NO" for nagios_enable in rc script.

  PR:		202556
  Submitted by:	John Marshall
  Sponsored by:	Absolight

Changes:
  head/net-mgmt/nagios4/Makefile
  head/net-mgmt/nagios4/files/nagios.in