The following commit broke the display of the guest name in the title for VNC displays: commit 61a1bc4dcbb056755fe96c5945f84c1312041059 Author: Pavel Grunt <pgrunt@xxxxxxxxxx> Date: Wed Apr 15 13:50:35 2015 +0200 session-vnc: Set window for display to avoid gtk-vnc v0.3.8 crash The VNC display widget of gtk-vnc v0.3.8 needs a window at the moment The problem is that this causes the window to be associated with the display before the guest name is available. Thus when ensure_window_for_display() runs, the window is already configured and so it never invokes the logic to set the title. The fix is to unconditionally update the title in the ensure_window_for_display() method, even if the window already exists. Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx> --- src/virt-viewer-app.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c index f983db2..e8412ac 100644 --- a/src/virt-viewer-app.c +++ b/src/virt-viewer-app.c @@ -898,7 +898,6 @@ ensure_window_for_display(VirtViewerApp *self, VirtViewerDisplay *display) if (l && virt_viewer_window_get_display(VIRT_VIEWER_WINDOW(l->data)) == NULL) { win = VIRT_VIEWER_WINDOW(l->data); g_debug("Found a window without a display, reusing for display #%d", nth); - virt_viewer_app_set_window_subtitle(self, win, nth); if (self->priv->fullscreen && !self->priv->kiosk) app_window_try_fullscreen(self, win, nth); } else { @@ -907,6 +906,7 @@ ensure_window_for_display(VirtViewerApp *self, VirtViewerDisplay *display) virt_viewer_window_set_display(win, display); } + virt_viewer_app_set_window_subtitle(self, win, nth); return win; } -- 2.7.4 _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list