Re: [libvirt] [PATCH] virsh: introduce name-uuid for list command
At 2016-06-20 21:38:42, "Peter Krempa" <pkrempa(a)redhat.com> wrote:
On Mon, Jun 20, 2016 at 19:22:45 +0800, Chen Hanxiao wrote:
>
> At 2016-06-15 17:36:05, "Chen Hanxiao" <chen_han_xiao(a)126.com>
wrote:
> >From: Chen Hanxiao <chenhanxiao(a)gmail.com>
> >
> >This patch will show both name and UUID of domain:
> >
> > Id Name State UUID
> >
---------------------------------------------------------------------------------------
> > 3 f23 running
918f1dd6-b19f-412b-ba17-d113bad89af8
> >
> >Signed-off-by: Chen Hanxiao <chenhanxiao(a)gmail.com>
> >---
>
> ping?
Jan already provided a review. I agree with his view. Allowing both
--name --uuid is better than the new argument.
Hi,
My only concern is that why we forbid --uuid with --name at the same time at the
begining.
If we let them come together, does that bring some backcompat issues?
Regards,
- Chen