[Spice-devel] [PATCH spice-gtk v2 09/15] desktop: Do not try org.gnome.SessionManager on Windows or Mac

Frediano Ziglio fziglio at redhat.com
Wed Jan 9 17:19:41 UTC 2019


> 
> From: Marc-André Lureau <marcandre.lureau at redhat.com>
> 
> DBus support is available on those systems, yet it doesn't make sense
> to attempt to connect to GNOME session manager.
> 
> Fixes:
> https://gitlab.freedesktop.org/spice/spice-gtk/issues/81
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>

Thanks, faster than light!

> ---
>  src/desktop-integration.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/src/desktop-integration.c b/src/desktop-integration.c
> index 7c433bb3..37d1791e 100644
> --- a/src/desktop-integration.c
> +++ b/src/desktop-integration.c
> @@ -160,8 +160,12 @@ static void
> spice_desktop_integration_init(SpiceDesktopIntegration *self)
>      priv = spice_desktop_integration_get_instance_private(self);
>      self->priv = priv;
>  
> -    if (!gnome_integration_init(self))
> -       g_warning("Warning no automount-inhibiting implementation
> available");
> +#if defined(G_OS_UNIX) && !__APPLE__
> +    if (gnome_integration_init(self))
> +        return;

style: missing brackets

this generates on Windows:

desktop-integration.c:56:17: error: 'gnome_integration_init' defined but not used [-Werror=unused-function]
 static gboolean gnome_integration_init(SpiceDesktopIntegration *self)
                 ^~~~~~~~~~~~~~~~~~~~~~

> +#endif
> +
> +    g_warning("Warning no automount-inhibiting implementation available");
>  }
>  
>  static void spice_desktop_integration_dispose(GObject *gobject)

Frediano


More information about the Spice-devel mailing list