[virt-tools-list] [PATCH virt-viewer] Do not show twice error dialog when no VM are found

Fabiano Fidêncio fabiano at fidencio.org
Sat Nov 29 19:56:51 UTC 2014


On Sat, Nov 29, 2014 at 4:12 PM, Marc-André Lureau
<marcandre.lureau at gmail.com> wrote:
> When running virt-viewer without argument, and no VM are found, you get
> two error dialogs. Only one is enough.
> ---
>  src/virt-viewer.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/src/virt-viewer.c b/src/virt-viewer.c
> index 16b90fa..193ef3a 100644
> --- a/src/virt-viewer.c
> +++ b/src/virt-viewer.c
> @@ -663,7 +663,6 @@ virt_viewer_initial_connect(VirtViewerApp *app, GError **error)
>          } else {
>              dom = choose_vm(&priv->domkey, priv->conn, &err);
>              if (dom == NULL && err != NULL) {
> -                virt_viewer_app_simple_message_dialog(app, err->message);
>                  goto cleanup;
>              }
>          }
> --
> 2.1.0
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list

ACK!

-- 
Fabiano Fidêncio




More information about the virt-tools-list mailing list