Bug#318664: [Pkg-nagios-devel] Bug#285554: acknowledged by developer (check_smtp update)

[Jeroen van Wolffelaar]

On Tue, Oct 11, 2005 at 04:03:12AM -0400, sean finney wrote:
> hi jeroen, steve,
> 
> if i understand correctly, there are two issues:
> 
> - offset of zero being treated as an error
> - usage of ntpdate
> 
> for the former, i believe this issue is resolved in the latest version
> of ntp.  for the latter, is using ntpdate really a problem?  jeroen:
> you claim that ntpdate is deprecated, but is it?  afaik it's still quite
> useful in many situations, including this case.

Steve wrote "... since ntpdate is deprecated ..." in the initial bug
submission, I just copied that. I don't know, something the NTP maintainer
knows more about of course (or maybe it's documented somewhere, didn't really
look either).
 
> On Sun, Oct 09, 2005 at 11:27:53PM +0200, Jeroen van Wolffelaar wrote:
> > gathering from ntpdate output to gathering the same data from ntpq output. If
> > ntpdate is really deprecated, something to pass along upstream, I'd say (true
> > in general for these bugs, I might add).
> 
> you're talking to someone who's also on upstream :)  in fact, i'd much
> rather deal with bugs in the debian bts then the POS sourceforge
> tracker, and then migrate them to upstream cvs in one big batch later on.

Ah, I see, didn't know that :). Thanks for your upstream work then, in that
case!

> > That said, my patch in #268044 should still apply manually -- while the regex
> > has changed, it became stricter rather than more lenient, and the point stands
> > that the exact format of each column really doesn't matter and only makes
> > things more fragile, and only the contents of those columns that are actually
> > used matter (2, 4 and 11). So also the current regex can be replaced by the
> > one I proposed in #268044.
> 
> i'll include this patch in the next update.  

Spiff.

--Jeroen

-- 
Jeroen van Wolffelaar
Jeroen at wolffelaar.nl (also for Jabber & MSN; ICQ: 33944357)
http://Jeroen.A-Eskwadraat.nl