[Nagiosplug-devel] 1.4.10 Release

Matthias Eble matthias.eble at mailing.kaufland-informationssysteme.com
Wed Sep 26 08:45:00 CEST 2007


>> Maybe someone wants to fix option processing in check_radius before
>> releasing. I'd also recommend to change or drop positional argument
>> support for check_radius. It's just too confusing if you ask me.
> 
> Indeed.  Does anyone have an idea why it was added in the first place?
> The first revision in SVN already has it, and there's no comment in the
> code.  It's not documented in the --help output, so I'd tend to simply
> removing it.  Any objections against the attached patch (which also
> checks whether all required arguments have been specified as suggested
> by Thomas in #1218438)?
> 

Hi Holger,

I'd prefer to remove the entire positional block, too.
So if you ask me, it should be committed, since it prevents the plugin 
from working as expected.

Matthias




More information about the Devel mailing list