Re: [libvirt] [PATCH 1/1] Fix incorrect implication about list options

[Eric Blake <eblake redhat com>]

On 07/25/2011 09:20 PM, Dave Allan wrote:
> The description of the list command seemed to suggest that it could
> take a set of domains as an argument, which is not correct in the
> current HEAD.  If virsh list is intended to take a list of domains,
> then this patch should be NAK'd and a bug opened against virsh list.
>
> Reported by hachi on #virt
>
> v2:
>
> Change language to include transient domains

> +++ b/tools/virsh.pod
> @@ -268,7 +268,7 @@ Inject NMI to the guest.
>
>   =item B<list>  [I<--inactive>  | I<--all>]
>
> -Prints information about one or more domains.  If no domains are
> +Prints information about existing domains.  If no options are
>   specified it prints out information about running domains.

ACK and applied.

--
Eric Blake   eblake redhat com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org