On Wed, May 7, 2008 at 2:32 PM, Max <<a href="mailto:perldork@webwizarddesign.com">perldork@webwizarddesign.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Wed, May 7, 2008 at 7:17 AM, Morten Guldager<br>
<<a href="mailto:morten.guldager@gmail.com">morten.guldager@gmail.com</a>> wrote:<br>
> 'Aloha!<br>
><br>
> I have found what looks like a bug in the file nagios-3.0.1/<a href="http://daemon-init.in" target="_blank">daemon-init.in</a><br>
> Find the function killproc_nagios<br>
><br>
> Looks like something is missing.... At least mine looks like this straight<br>
> form the tar-ball:<br>
><br>
> killproc_nagios ()<br>
> {<br>
><br>
>         kill $2 $NagiosPID<br>
><br>
> }<br>
><br>
> What do you think?<br>
<br>
</div>it is strange code :) .. $1 looks like it is always nagios and it is<br>
always ignored, $2 is the signal to send and that is used properly ..<br>
<br>
example from script ..<br>
<br>
 if status_nagios > /dev/null; then<br>
                    printf "Reloading nagios configuration..."<br>
                    killproc_nagios nagios -HUP<br>
                    echo "done"<br>
                else<br>
<br>
the script could use some syntax clean up for sure :) but that<br>
function looks like it works correctly even though it is strange to<br>
have kill_proc always passed an argument that is ignored.</blockquote><div><br>Agreed! (after reading the code a few times more...) <br></div><br></div><br>/Morten %-)<br>