On Dec 8, 2016, at 8:24 AM, Miroslav Lichvar <mlich...@redhat.com> wrote:
> On Thu, Dec 08, 2016 at 07:59:24AM -0600, Lonnie Abelbeck wrote:
>> On Dec 8, 2016, at 12:36 AM, Miroslav Lichvar <mlich...@redhat.com> wrote:
>>> Thanks, Lonnie. This is useful. Would it be possible to parse the list
>>> of commands from the usage text instead of maintaining a separate
>>> list? The chronyc support in bash-completition works like that.
>> I presume you meant the "help" output ... Yes, I briefly looked at that,
>> probably more code would be required than the data size of command_names,
>> and the command_names entries can be tweaked specifically for
>> My thought is to keep things simple (as this patch does) and have a separate
>> tab-completion list.
>> Possibly, down the road. give_help() text, process_line() parsing matches,
>> and tab-completion text could all be in one structure, but probably not
>> worth stirring-the-code over.
> Ok, fair enough.
> The patch looks good to me and it seems to be working nicely. I'll
> just make the strings const and move them so they are closer to the
> help text and it's less likely to update one without the other. I hope
> that's ok with you.
Yes, good idea.
Thanks for including this patch upstream.
To unsubscribe email chrony-users-requ...@chrony.tuxfamily.org
with "unsubscribe" in the subject.
For help email chrony-users-requ...@chrony.tuxfamily.org
with "help" in the subject.
Trouble? Email listmas...@chrony.tuxfamily.org.