Page 1 of 1

Re: [Nagios-devel] Patch for leading whitespace on comments

Posted: Fri Jan 19, 2007 1:34 pm
by Guest
Tobias Klausmann wrote:
> Hi!
>
> (Sent this to the wrong list initially. Sorry 'bout that)
>
> As of 2.6, Nagios recognizes leading whitespace only in the main
> config file. The attached patch fixes this.
>
> To illustrate, this piece of config does not work currently:
>
> define host {
> host_name eric.schwarzvogel.de
> alias eric.schwarzvogel.de
> address 194.97.4.250
> #check_command check_host_alive
> check_command check_host_new
> max_check_attempts 3
> retain_status_information 1
> contact_groups Privat Klausmann
> notification_interval 30
> notification_period 24x7
> notification_options d,u,r
> passive_checks_enabled 1
> }
>
> The reason is the commented-out check_command line. It would work
> like this:
>
> define host {
> ...
> # check_command check_host_alive
> check_command check_host_new
> ...
> }
>
> This is because (for non-main config files), Nagios first checks
> if the first char is "#" and *then* strips the line. For the main
> config file, this happens the other way around.
>
> I've attached a patch that makes this behaviour more consistent.
> I believe the check for '\0', '\n' etc. could be simplified to,
> *if* strip() removes some of the chars. I tried to be
> non-invasive as possible, so I didn't do that.
>
> I'd like it if this patch makes it into the 2.x series.
>
> Regards,
> Tobias

Good suggestion. I'll actually fix the problem in xdata/xodtemplate.c
to make the example you gave above work without problems. Will be in
CVS shortly.


Ethan Galstad,
Nagios Developer
---
Email: [email protected]
Website: http://www.nagios.org





This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]