Re: [PATCH v2 0/3] Introduce aliases for virt-admin's srv-* commands

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

 



On 13/09/16 17:11, Erik Skultety wrote:
> the original version:
> https://www.redhat.com/archives/libvir-list/2016-September/msg00312.html
> 
> since v1:
>     - tweaked the virsh-self-test so that it also checks the aliased commands
>     instead of skipping them (since there was a good reason for that before the
>     changes this series introduces)
>     - patches 2-3 remained untouched
> 
> Erik Skultety (3):
>   virt-admin: Tweak command parsing logic so that aliases point to new
>     commands
>   virt-admin: Add some command aliases to provide syntax sugar over ugly
>     commands
>   virt-admin: Replace the (now) aliases with new command names in the
>     man page
> 
>  tools/virsh-nodedev.c |  6 ++----
>  tools/virsh.c         | 10 ++++++----
>  tools/virsh.pod       |  2 --
>  tools/virt-admin.c    | 24 ++++++++++++++++++++++++
>  tools/virt-admin.pod  | 30 +++++++++++++++---------------
>  tools/vsh.c           |  6 ++++++
>  tools/vsh.h           |  1 +
>  7 files changed, 54 insertions(+), 25 deletions(-)
> 

SNACK. Having an error like "internal error: bad command options" would
be nice indeed - thanks Michal [1] - which could be achieved by a tiny
refactor, also fixing a semantically incorrect check that can cause a
segfault on missing '.info' as well, pretty much the same way as the
missing '.alias' does.

[1] https://www.redhat.com/archives/libvir-list/2016-September/msg00386.html

Erik

--
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]