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

Martin Kletzander mkletzan at redhat.com
Thu Jan 2 09:39:11 UTC 2014


On Tue, Dec 24, 2013 at 10:34:37AM +0100, Giuseppe Scrivano wrote:
> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1038496
> Signed-off-by: Giuseppe Scrivano <gscrivan at redhat.com>
> ---
>  virt-manager | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Trivial, ACK.

> 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
> --
> 1.8.3.1
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20140102/eafec1fc/attachment.sig>


More information about the virt-tools-list mailing list