On Sat, Sep 26, 2015 at 11:42:11AM +0100, Raf Czlonka wrote:
> On Sat, Sep 26, 2015 at 09:52:39AM BST, Stuart Henderson wrote:
> > On 2015/09/26 07:44, Raf Czlonka wrote:
> > > On Fri, Sep 25, 2015 at 10:14:03PM BST, Rob Pierce wrote:
> > > 
> > > > Update diff below referencing the correct man page sections and with 
> > > > sentenced
> > > > starting on a new line.
> > > 
> > > Hi Rob,
> > > 
> > > > 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 02:23:33 -0000
> > > > @@ -124,22 +124,30 @@ Furthermore, the superuser's
> > > >  should never contain the current directory
> > > >  .Pq Dq \&. .
> > > >  .Ss System date
> > > > -Check the system date with the
> > > > +.Xr ntpd 8
> > > > +is now enabled by default to automatically synchronize clocks with 
> > > > remote
> > > 
> > > If I may. I'd simply use what's already there with slight modification:
> > > 
> > > +is used to automatically synchronize clocks with remote NTP servers.
> > 
> > Agreed. In particular "is now" doesn't seem helpful for afterboot,
> > people reading this don't need to know about what we used to do.
> 
> That's exactly what I was thinking, too.
> 
> > > > +NTP servers.
> > > > +You can use
> > > > +.Xr ntpctl 8
> > > > +to check the status.
> > > > +You can also search
> > > > +.Pa /var/log/daemon
> > > > +for
> > > > +.Xr ntpd 8
> > > > +related events.
> > > > +To change the time servers or other related settings see
> > > > +.Xr ntpd.conf 5 .
> > > > +.Pp
> > > 
> > > IMVHO, too much info about ntpd. These do not belong here at all - ntpd
> > > "just works" and if one is to find out more about it - they should read
> > > the aforementioned manual page.
> > 
> > I think the ntpctl and /var/log/daemon references are unnecessary, but
> > since the ability to set a timeserver was removed from the installer
> > it might be a good idea to point people in the right direction to do
> > that (i.e. mention ntpd.conf). If I have a bunch of machines and want
> > to correlate times of logged events between them, I do not want them
> > all picking different pool.ntp.org servers, I want them to look at
> > a single or small number of sources, usually ones which I control
> > myself.
> 
> Fair point.
> 
> > > > +You can check the system date using the
> > > >  .Xr date 1
> > > >  command.
> > > > -If needed, change the date, and/or change the symbolic link of
> > > 
> > > This was fine as it is, with how to *change* the date example below.
> > > 
> > > > +If needed, change the symbolic link of
> > > >  .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,
> > > 
> > > Raf
> > > 
> > 
> > I'd be tempted to make the example use something from March 2038 :-)
> 
> :^)
> 
> > If not that, then at least use a contemporary date, a 1999 example
> > makes the page look outdated.
> 
> Either one would be fine, IMHO.
> 
> Raf

Here is an updated diff. I wasn't sure whether to go with "timeserver" or
"time server" so I checked the install script (install.sub) and went with
"NTP server".

I was tempted to change the date reference, but thought this is likely already
a date that has significance for someone out there.

I really like the reference to Canadian Atlantic Standard Time...

Rob

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 16:22:07 -0000
@@ -124,6 +124,14 @@ Furthermore, the superuser's
 should never contain the current directory
 .Pq Dq \&. .
 .Ss System date
+.Xr ntpd 8
+is used to automatically synchronize clocks with remote NTP servers.
+You can use
+.Xr ntpctl 8
+to check the status.
+To change the NTP server see
+.Xr ntpd.conf 5 .
+.Pp
 Check the system date with the
 .Xr date 1
 command.
@@ -132,14 +140,11 @@ If needed, change the date, and/or chang
 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
+Set the current date to January 27th, 2015 3:04pm:
+.Dl # date 201501271504
 .Pp
 Set the time zone to Atlantic Standard Time:
 .Dl # ln -fs /usr/share/zoneinfo/Canada/Atlantic /etc/localtime

Reply via email to