Jason McIntyre <j...@kerhand.co.uk> writes:

> On Fri, Feb 13, 2015 at 09:49:58PM -0600, Adam Thompson wrote:
>> Fix a minor typo in the ntpctl help output, and same semantic mistake in the
>> manpage.  Currently says (to be pedantic) that '-s' only works with 'all',
>> which is not the case: '-s' is required with all options.
>> Apologies if I've got the patch format wrong, I don't do this often...
>> -Adam
>> 
>> 
>> --- ntpd.c_r1.92        Fri Feb 13 21:43:39 2015
>> +++ ntpd.c      Fri Feb 13 21:43:49 2015
>> @@ -90,7 +90,7 @@
>> 
>>         if (strcmp(__progname, "ntpctl") == 0)
>>                 fprintf(stderr,
>> -                   "usage: ntpctl [-s all | peers | Sensors | status]\n");
>> +                   "usage: ntpctl -s [all | peers | Sensors | status]\n");
>>         else
>>                 fprintf(stderr, "usage: %s [-dnSsv] [-f file]\n",
>>                     __progname);
>> 
>> 
>> --- ntpctl.8_r1.7       Fri Feb 13 21:39:53 2015
>> +++ ntpctl.8    Fri Feb 13 21:40:32 2015
>> @@ -22,7 +22,8 @@
>>  .Nd control the Network Time Protocol daemon
>>  .Sh SYNOPSIS
>>  .Nm ntpctl
>> -.Op Fl s Cm all | peers | Sensors | status
>> +.Fl s
>> +.Op Cm all | peers | Sensors | status
>>  .Sh DESCRIPTION
>>  The
>>  .Nm
>> 
>
> i didn;t realise that -s was mandatory. pretty yucky that we have a
> utility with only one option, and it's not optional ;(

Maybe ntpctl will provide more options/features one day, who knows? :)

-- 
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

Reply via email to