showall - peers display
Mark Atwood
fallenpegasus at gmail.com
Mon Dec 19 20:44:33 UTC 2016
If Hal thinks it should go, it should go. Thank you Hal!
On Mon, Dec 19, 2016 at 4:26 AM Eric S. Raymond <esr at thyrsus.com> wrote:
> Hal Murray <hmurray at megapathdsl.net>:
> >
> > There is a showall parameter to the worker printout code for the peers
> > display in ntpq and ntpmon.
> >
> > ntpmon has the s keyboard command to toggle it. ntpq has lpeers and
> > lassociations.
> >
> > Short version: I vote we nuke it - general cruft reduction.
> >
> > Long version:
> >
> > I've never found a good use for it. I have been confused because peers
> hid
> > info that I was looking for.
> >
> > Nuking it will have subtle changes in the UI. We can leave the old ntpq
> > commands around as aliases, but I'd vote for removing them.
> >
> > I just set the default in ntpmon to True.
> > I hacked peers and assoc in ntpq to use True which makes them like
> lpeers and
> > lassoc. There are also lopeers and lpassoc.
>
> In principle, I'm not opposed to this as a de-crufting of the design.
>
> In practice, I wonder if the cleanup buys us more than appearing not
> to care about backwards compatibility costs us.
>
> However, I'm not sure enough this is a problem to revert Hal's change.
> And if he wants to nuke ntpmon's 's' command, I won't stop him.
>
> Hal is taking responsibility for ntpq/ntpmon, therefore he gets the
> authority to make these decisions unless I judge that he is making a
> serious mistake - not a contigency I am ever expecting.
>
> I would like to see other people step up like this (Gary has with
> ntpviz, for which I am also grateful). It isn't healthy or
> sustainable for me to do as large a percentage of the work and the
> routine decision-making as I have up to now.
> --
> <a href="http://www.catb.org/~esr/">Eric S. Raymond</a>
> _______________________________________________
> devel mailing list
> devel at ntpsec.org
> http://lists.ntpsec.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20161219/7f1a1524/attachment.html>
More information about the devel
mailing list