<div dir="ltr"><div dir="ltr"><div dir="ltr">On Sat, Mar 16, 2019 at 11:33 AM Achim Gratz via devel <<a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>As already briefly mentioned on IRC, the way the mode 6 messaging was<br>changed to do no filtering at all anymore doesn't really work nicely.<br>To wit here are two otherwise identical systems, one with and the other<br>without those changes getting asked for their system variables:<br></blockquote><div>:::snip::: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Can the offending two commits (ee4ce108 removes the filtering, 4c5a32fb<br>removes the flag bits the filtering was based on) be reverted and<br>something more sensible be discussed here and then implemented.  And no,<br>neither is the first commit a "Feature", quite the contrary, nor the<br>second a "Refactor".  Both remove a feature or code, don't add or<br>re-arrange.<br>
<br>I haven't checked extensively if the list feature still works that<br>allows to flag a user-selected set of variables for output, which can<br>then be used with rl / cl variants of the cv / rv commands; the basics<br>still seem to be there.  One of the problems is that there is only one<br>list that gets applied to both the system and the clock variables.  So a<br>useful feature would be if such lists could be named.  If that feature<br>works, then predefined lists containing all system and clock variables<br>(the output of a query for sys_var_list on the appropriate association<br>ID) could be provided to achieve the "no filters" output more easily and<br>directly.  Alternatively, extra variants for rv and cv could be provided<br>to do the two queries for the user.<br>
<br>
<br>Regards,<br>Achim.<br></blockquote><div>:::snip:::</div><div><br></div><div>(re)posted to <a href="https://gitlab.com/NTPsec/ntpsec/merge_requests/963#note_151166759">https://gitlab.com/NTPsec/ntpsec/merge_requests/963#note_151166759</a></div><div>which is probably not what you want. <br></div></div></div></div>