> On Thu, Nov 30, 2017 at 6:24 PM, Frediano Ziglio < fziglio@xxxxxxxxxx > > wrote: > > > On Wed, Nov 29, 2017 at 5:03 PM, Christophe Fergeau < cfergeau@xxxxxxxxxx > > > > > > > > wrote: > > > > > On Mon, Nov 13, 2017 at 06:50:05PM +0100, Jakub Janků wrote: > > > > >> This is a preparatory patch for GTK+ vdagent redesign. > > > > >> > > > > >> Since most of the vdagent code still uses X11 lib, > > > > >> force GDK to use X11 backend by calling > > > > >> gdk_set_allowed_backends("x11"). > > > > > > > > > > What happens if the agent tries to use gtk+, and this call is not > > > > > present? > > > > > > > > > > Christophe > > > > I'll fix this in the next version, I was waiting for reviews of the other > > > > patches (2/3, 3/3). > > > > gdk_set_allowed_backends() was added in v3.10. Prior to this version, > > > > GDK_IS_X11_DISPLAY() should be used after gtk_init() to make sure we are > > > > using X11. > > > > Or would anyone be against requiring GTK+ v3.10? > > > > Cheers, > > > > Jakub > > > According to > > https://github.com/FreeRDP/Remmina/wiki/GTK-versions-on-various-distributions > > > no RHEL7 then (wheezy and precise are in theory supported even in 2018). > > OK then, I will use GDK_IS_X11_DISPLAY() and terminate the program when this > check fails. > > Should not we have > > > PKG_CHECK_MODULES([GTK], [gtk+-3.0 >= 3.10]) > > > ? > > What do you mean by this? > Jakub Yes, the 2 suggestions seem to conflict. I'm not sure about the page I pointed out. For instance I just checked RHEL 7.4 and there's gtk 3.22 so is fine, maybe the page is referring at the first release, not sure if we should support still RHEL 7.0. Maybe we can assume there's at least gtk 3.10. -- Frediano OT: Please try to reply using text format, not html. _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel