[Nagiosplug-devel] 1.4.10 Release
Holger Weiss
holger at CIS.FU-Berlin.DE
Wed Sep 26 13:00:52 CEST 2007
* Matthias Eble <matthias.eble at mailing.kaufland-informationssysteme.com> [2007-09-26 08:45]:
> >> 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)?
>
> 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.
Okay, done.
Thanks, Holger
More information about the Devel
mailing list