Re: [PATCH virt-manager] virt-manager: use DISPLAY in the error message only when it is set

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

 



Cole Robinson <crobinso@xxxxxxxxxx> writes:

> On 12/24/2013 04:34 AM, Giuseppe Scrivano wrote:
>> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1038496
>> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
>> ---
>>  virt-manager | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/virt-manager b/virt-manager
>> index eab9675..1f58655 100755
>> --- a/virt-manager
>> +++ b/virt-manager
>> @@ -214,7 +214,7 @@ def main():
>>          # it avoids the issue
>>          display = os.environ.get("DISPLAY", "")
>>          msg = str(e)
>> -        if not display:
>> +        if display:
>>              msg += ": Could not open display: %s" % display
>>          logging.debug("".join(traceback.format_exc()))
>>          print msg
>> 
>
> ACK

Pushed now.

Giuseppe

_______________________________________________
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