Re: [PATCH] Update warning msg in virt-viewer's window

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

 



On Tue, Dec 27, 2016 at 10:57 AM, Snir Sheriber <ssheribe@xxxxxxxxxx> wrote:
> Warning "Display can only be attached through libvirt with --attach" is
> now appears also in virt-viewer's window (as in the cmd prompt) when
> virt-viewer is waiting without --attach and then the guest starts
> (instead of showing "Waiting for guest domain to start")
>
> rhbz#1386630

The idea is good but I here is not the right place to report the error..
I'd say we probably want to do all the error reporting on
app_activate() instead.

> ---
>  src/virt-viewer-app.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index bf32315..01392f3 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -1219,6 +1219,7 @@ virt_viewer_app_default_activate(VirtViewerApp *self, GError **error)
>      } else {
>          g_set_error_literal(error, VIRT_VIEWER_ERROR, VIRT_VIEWER_ERROR_FAILED,
>                              _("Display can only be attached through libvirt with --attach"));
> +        virt_viewer_app_show_status(self, _("Display can only be attached through libvirt with --attach"));
>     }
>
>      return FALSE;
> --
> 2.9.3
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/virt-tools-list



-- 
Fabiano Fidêncio

_______________________________________________
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