Re: [PATCH 0/5] Another attempt at improving virsh autocompletion

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/29/2014 08:36 PM, Solly Ross wrote:
> This patch set represents yet another attempt to improve the virsh
> autocompletion.  When writing this patch, I tried to take into account
> the comments on the previous attempt at autocompletion by Tomas Meszaros.

Nice - I'm hoping you can get it all the way!

> 
> Specifically, it was reccomended that any completion use the existing
> methods that virsh has in place to parse the partial line being completed.

Also recommended was to enhance the testsuite to prove that we have
implemented things correctly, as well as to add a new virsh command that
can be reused from the shell to make writing the shell completer easier
(the two are intricately related - our testsuite will use the same
completer as the shell would use).  However...


>  outgoing/0000-cover-letter.patch                   |   68 +
>  ...prove-virsh-autocompletion-extract-parser.patch |  428 ++++++
>  ...prove-virsh-autocompletion-base-framework.patch |  278 ++++
>  ...ve-virsh-autocompletion-global-ctl-object.patch |  116 ++
>  ...ove-virsh-autocompletion-domain-completer.patch | 1479 ++++++++++++++++++++
>  ...virsh-autocompletion-enum-completer-macro.patch |  131 ++
>  tools/Makefile.am                                  |    3 +-
>  tools/virsh-completer.c                            |  353 +++++
>  tools/virsh-completer.h                            |   86 ++
>  tools/virsh-domain-monitor.c                       |  287 +---
>  tools/virsh-domain.c                               |   72 +
>  tools/virsh-snapshot.c                             |   11 +
>  tools/virsh.c                                      |  517 +++++--
>  tools/virsh.h                                      |   13 +
>  14 files changed, 3447 insertions(+), 395 deletions(-)

...I see no mention of a testsuite change.  While I don't mind reviewing
the patches, I'm unwilling to accept them into the tree without a
testsuite and a 'virsh complete' command that the shell can use to hook
into the completion engine.

https://www.redhat.com/archives/libvir-list/2013-September/msg01029.html

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]