[libvirt] [PATCH 2/2] Indicate freecell --cellno is optional
Eric Blake
eblake at redhat.com
Thu Mar 22 20:10:30 UTC 2012
On 03/22/2012 01:59 PM, Dave Allan wrote:
> ---
> tools/virsh.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index 9e5c9b2..d9cff0c 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -4742,7 +4742,7 @@ static const vshCmdInfo info_freecell[] = {
> };
>
> static const vshCmdOptDef opts_freecell[] = {
> - {"cellno", VSH_OT_INT, 0, N_("NUMA cell number")},
> + {"cellno", VSH_OT_INT, VSH_OFLAG_REQ, N_("NUMA cell number")},
NACK. VSH_OFLAG_REQ means required, absence of that flag (ie. using 0
for the flag) means optional. This patch would break the command by
requiring a --cellno argument, even with --all.
This is the current 'virsh help freecell' output, without your patch:
$ virsh help freecell
NAME
freecell - NUMA free memory
SYNOPSIS
freecell [--cellno <number>] [--all]
DESCRIPTION
display available free memory for the NUMA cell.
OPTIONS
--cellno <number> NUMA cell number
--all show free memory for all NUMA cells
It shows that both --cellno and --all are optional; however, what it
does not show (and cannot show, without a lot more work throughout
virsh), is the notion of mutual exclusion (that is, there is no trivial
way to make virsh help output the {} operators to show the alternation
that the virsh.pod has by hand).
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120322/15fe1f6c/attachment-0001.sig>
More information about the libvir-list
mailing list