> On 30/06/2017, at 11:37 PM, Miroslav Lichvar <mlich...@redhat.com> wrote:
> 
> This looks great.

Thanks.

> 
> Can you please also update documentation of the -r option and the
> dumpdir, corrtimeratio, leapsecmode, rtcsync, maxdrift, and maxslewrate
> directives? Where all supported systems would be listed, it now may be
> better to say everything except macOS before 10.13.

OK - I'll do that.

> Another possibility is to replace this with a new case switch inside
> the Darwin) case which would simply define a macro for versions 10.13
> and newer. SunOS-sun4* used to be some version specific configuration.

Yes - I think this is the cleanest solution. Then there is no need to check for 
<sys/timex.h>. If it is missing then the tool chain is broken.

Bryan Christianson
br...@whatroute.net




--
To unsubscribe email chrony-dev-requ...@chrony.tuxfamily.org with "unsubscribe" 
in the subject.
For help email chrony-dev-requ...@chrony.tuxfamily.org with "help" in the 
subject.
Trouble?  Email listmas...@chrony.tuxfamily.org.

Reply via email to