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

Reply via email to