[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