[libvirt] [PATCH v2 1/2] virsh: allow both --uuid and --name at same cmd

[Chen Hanxiao]

At 2016-06-29 05:36:25, "John Ferlan" <jferlan at redhat.com> wrote:
>
>
>On 06/24/2016 12:44 PM, Chen Hanxiao wrote:
>> From: Chen Hanxiao <chenhanxiao at gmail.com>
>> 
>> virsh # list  --uuid --name
>> c7    49c765a0-25e7-40d0-964f-dac99724b32c
>> f23   918f1dd6-b19f-412b-ba17-d113bad89af8
>> 
>
>The actual output is reversed - perhaps a late change not properly
>reflected in the /commit message...
>
># virsh list --all --name --uuid
>56c1f811-3ffc-4363-b2d9-06bdc9fbbe2b dom1
>abeb5797-df38-42c7-9265-c3359d409ab5 dom2
>
>> Signed-off-by: Chen Hanxiao <chenhanxiao at gmail.com>
>> ---
>> v2: drop uuid-name option, enable both --uuid and --name
>> 
>>  tools/virsh-domain-monitor.c | 14 ++++++++------
>>  1 file changed, 8 insertions(+), 6 deletions(-)
>> 
>
>virsh.pod needs be changed to indicate, right after "If I<--name> is
>specified, domain names are printed instead of the table formatted one
>per line. If I<--uuid> is specified domain's UUID's are printed instead
>of names."
>
>The following is a suggestion and would replace the remainder of that
>paragraph:
>
>
>If both I<--name> and I<--uuid> are specified, domain UUID's and names
>are printed side by side without any header. Flag I<--table> specifies
>that the legacy table-formatted output should be used. This is the
>default if neither I<--name> nor I<--uuid> are specified. Options
>I<--uuid> and I<--name> are mutually exclusive if option I<--table> is
>specified.
>
>

Sure, v3 will add virsh.pod.

Regards,
- Chen