----- Original Message -----
> From: "Michael Reed" <m.r...@mykolab.com>
> To: "Rob Pierce" <r...@2keys.ca>, "tech" <tech@openbsd.org>
> Sent: Friday, September 25, 2015 9:12:25 PM
> Subject: Re: afterboot.8 to reference ntpd(1) enabled by default

> Hi Rob,
> 
> No comment on if this is a good idea or not as I'm not an OpenBSD developer,
> but I left a few comments inline regarding the mdoc semantics.
> 
> On 09/25/15 16:57, Rob Pierce wrote:
>> Since ntpd(1) is now enabled by default do the following changes make sense?
>> 
>> Regards,
>> 
>> Index: afterboot.8
>> ===================================================================
>> RCS file: /cvs/src/share/man/man8/afterboot.8,v
>> retrieving revision 1.149
>> diff -u -p -r1.149 afterboot.8
>> --- afterboot.8      24 Sep 2015 15:07:55 -0000      1.149
>> +++ afterboot.8      26 Sep 2015 00:36:30 -0000
>> @@ -124,22 +124,28 @@ Furthermore, the superuser's
>>  should never contain the current directory
>>  .Pq Dq \&. .
>>  .Ss System date
>> -Check the system date with the
>> +.Xr ntpd 1
> 
> I think you meant section 8, not 1.
> The port textproc/igor can, among other things, automatically check for
> the existence of external references with the -x flag.
> 
>> +is now enabled by default to automatically synchronize clocks with remote
>> +NTP servers.  You can use
> 
> Sentences should start on a new line; not sure if it makes a difference in
> mandoc's output, but it's done in the vast majority of manuals (perhaps a
> stylistic choice).
> 
> In any case, igor can check for this too.
> 
>> +.Xr ntpctl 1
> 
> 1 -> 8
> 
>> +to check the status.
>> +You can also search
>> +.Pa /var/log/daemon
>> +for
>> +.Xr ntpd 1
> 
> 1 -> 8
> 
>> +related events.
>> +To change the time servers or other related settings see
>> +.Xr ntpd.conf 8 .
> 
> 8 -> 5
> 
>> +.Pp
>> +You can check the system date using the
>>  .Xr date 1
>> -command.
>> -If needed, change the date, and/or change the symbolic link of
>> +command.  If needed, change the symbolic link of
> 
> Sentences should start on a new line.
> 
>>  .Pa /etc/localtime
>>  to the correct time zone in the
>>  .Pa /usr/share/zoneinfo
>>  directory.
>> -Alternatively,
>> -.Xr ntpd 8
>> -can be used to automatically synchronize clocks with remote NTP servers.
>> -.Pp
>> -Examples:
>>  .Pp
>> -Set the current date to January 27th, 1999 3:04pm:
>> -.Dl # date 199901271504
>> +For example:
>>  .Pp
>>  Set the time zone to Atlantic Standard Time:
>>  .Dl # ln -fs /usr/share/zoneinfo/Canada/Atlantic /etc/localtime
>> 
> 
> Regards,
> Michael

Yikes. That was a sloppy one. Should stay away from submitting diffs on Friday 
nights!

I will resubmit.

Thanks.

Rob

Reply via email to