[Nagiosplug-devel] [ nagiosplug-Patches-1119917 ] Added ipv4/ipv6 options to check_ntp

SourceForge.net noreply at sourceforge.net
Fri Mar 4 13:52:35 CET 2005


Patches item #1119917, was opened at 2005-02-10 09:46
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1119917&group_id=29880

Category: Enhancement
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Submitted By: Seaworthy (mrseaworthy)
>Assigned to: Ton Voon (tonvoon)
Summary: Added ipv4/ipv6 options to check_ntp

Initial Comment:
The options -4 and -6 now specify the protocol to use.
This addition was made because the script failed in the 
original version when contacting a server over ipv6.
The enhancement makes use of the -46 options of 
ntpdate and ntpq.
The original version of check_ntp I used was 1.20 and I 
made this version 1.30.
Also i made some minor adjustments, like newlines and 
comments on the new options in the help output.


----------------------------------------------------------------------

>Comment By: Ton Voon (tonvoon)
Date: 2005-03-04 21:51

Message:
Logged In: YES 
user_id=664364

Merijn,

Thanks for the patch. This is useful, so it is now committed to CVS 
HEAD. I've made a change where the default is not to add either -4 nor -6 
for backward compatibility, but I can't test this from my home system. Can 
you confirm this works correctly.

Also, I've removed your copyright. As a rule, we don't put copyright into 
plugins, but we credit you in the CVS comments and in our list of 
contributors.

BTW, a unified diff (-u) or contextual diff (-c) is more helpful than the 
default line by line.

Ton

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397599&aid=1119917&group_id=29880




More information about the Devel mailing list