> > Hi > > On Wed, Jan 9, 2019 at 9:43 PM Marc-André Lureau > <marcandre.lureau@xxxxxxxxx> wrote: > > > > Hi > > > > On Wed, Jan 9, 2019 at 9:19 PM Frediano Ziglio <fziglio@xxxxxxxxxx> wrote: > > > > > > > > > > > From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx> > > > > > > > > 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@xxxxxxxxxx> > > > > > > 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) > > > ^~~~~~~~~~~~~~~~~~~~~~ > > > > Oh well, it is solved by the following patch, I can squash both. > > ack with the squash? > Better to send an updated patch, looks like more a merge than a simple squash. Looking after 10/15 it seems it would be reasonable to have multiple SpiceDesktopIntegration implementation (currently one dummy and a Gnome one). > > > > > > > > > +#endif > > > > + > > > > + g_warning("Warning no automount-inhibiting implementation > > > > available"); > > > > } > > > > > > > > static void spice_desktop_integration_dispose(GObject *gobject) > > > Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel