<div dir="ltr"> I have read ESR's writeup on our buglist, and agree with his assessments.  ..m<br><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 15, 2017 at 5:27 AM Hal Murray via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> * I need to work on #348: reverse function for restrict<br>
> * unpeer should be made to fully work from ntpq :config.  This one is mine too.<br>
<br>
There is a quirk tangled in this area.  I don't know if there is a bug for it.<br>
<br>
When the pool mode adds a server, if needed, it pokes a hole in the<br>
restrictions.<br>
<br>
We need to remove that hole when that server is removed.<br>
<br>
I think we need to add a new flag to indicate that the restrict slot was<br>
automatically added.<br>
<br>
Maybe we should add another flag to disable poking holes.  Maybe it's an<br>
enhancement rather than bug fix, but this would be the time to do it.<br>
<br>
-------<br>
<br>
I didn't see fixing ntpq retransmissions on your list.  (I'm still catching<br>
up so I might have missed it.)<br>
<br>
I think the way to fix this is to clean up the logging.<br>
<br>
I've never been particularly happy with the standard log level approach.<br>
Maybe it would make more sense if there was a description of what the levels<br>
were intended to cover.<br>
<br>
Some of the cruft may be my fault.  I hacked a lot of the logging to do what<br>
I needed when chasing some bug(s?).  I may have broken any plan that you had.<br>
<br>
Maybe we need log-to-file.<br>
<br>
In the context of fixing this bug, I think I would like a logging mode that<br>
showed the command line and the packets.  Reply packets can be verbose so<br>
maybe we need a switch/level for that.  Maybe we need another switch/level to<br>
show steps within long running commands.  Mumble.<br>
<br>
<br>
--<br>
These are my opinions.  I hate spam.<br>
<br>
<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@ntpsec.org" target="_blank">devel@ntpsec.org</a><br>
<a href="http://lists.ntpsec.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ntpsec.org/mailman/listinfo/devel</a><br>
</blockquote></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><p dir="ltr">Mark Atwood<br>
<a href="http://about.me/markatwood">http://about.me/markatwood</a><br>
+1-206-604-2198 Mobile & Signal</p>
</div>