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@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list