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

Reply via email to