Re: [virt-manager PATCH] virt-install: Make default graphics configurable

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

 



On 07/16/2013 02:10 AM, Martin Kletzander wrote:
> On 07/15/2013 09:57 PM, Cole Robinson wrote:
>> On 07/15/2013 04:00 AM, Martin Kletzander wrote:
>>> The '--default-graphics' option of 'setup.py configure' only affects
>>> virt-manager, but not virt-install.  This should be unified, so this
>>> patch checks whether default_graphics taken from the cli is supported
>>> and uses that option.
>>>
>>
>> ACK
>>
> 
> Thanks, pushed,
> 

Looks like it breaks the test suite. The change in cliconfig.py changes the
default if ./setup.py configure was never run, like on a fresh checkout. I
think that bit should be reverted, but we also want to hardcode a default for
this setting in the test suite so the results don't change depending on how
the user configured things. Can probably set it in tests/__init__.py, there's
precedent there.

- 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