Re: [virt-manager PATCH] virt-install: add "-c" as short option of "--connect"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/11/2014 09:22 AM, Chen Hanxiao wrote:
> 
> 
> On 02/11/2014 09:04 PM, Cole Robinson wrote:
>> On 02/11/2014 01:58 AM, Chen Hanxiao wrote:
>>> From: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx>
>>>
>>> Add "-c" as short option of "--connect".
>>> Replace original "-c" as "-cd" for "--cdrom"
>>>
>>> This will be much convenient if we operate
>>> non-default hypervisor.
>>>
>>> Signed-off-by: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx>
>>> ---
>>>   man/virt-install.pod | 2 +-
>>>   man/virt-xml.pod     | 2 +-
>>>   virt-install         | 7 ++-----
>>>   virtinst/cli.py      | 2 +-
>>>   4 files changed, 5 insertions(+), 8 deletions(-)
>>>
>>
>> This will break any pre-existing users of virt-install -c <cdrom>, so it's not
>> acceptable. We could make the -c shortcut available for the other tools, but
>> then things would be inconsistent. So I'd rather just leave it the way it is.
>>
> 
> How about give --connect a shortcut?
> It's not so convenient if using virt-xml editing LXC domain.
> 

We should use any other shortcut besides '-c' because that's what all other
libvirt apps use. unfortunately virt-install with its -c shortcut existed
before --connect was widespread.

I'll retract my objection about adding -c shortcut for virt-xml. But we can't
do it for virt-install.

- Cole

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux