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