Re: [PATCH v3 2/2] Lower gtk+ requirement to 3.10

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/22/2016 08:33 AM, Marc-André Lureau wrote:
> Hi
> 
> ----- Original Message -----
>> Hi,
>>
>>> diff --git a/src/spice-widget-egl.c b/src/spice-widget-egl.c
>>> index 1b3cd07..a9bea52 100644
>>> --- a/src/spice-widget-egl.c
>>> +++ b/src/spice-widget-egl.c
>>> @@ -29,7 +29,9 @@
>>>  #include <libdrm/drm_fourcc.h>
>>>  
>>>  #include <gdk/gdkx.h>
>>> +#if GTK_CHECK_VERSION(3,16,0)
>>>  #include <gdk/gdkwayland.h>
>>> +#endif
>>
>> This breaks builds on Debian, which has 3.14.  You get a failure on line
>> 208 because GDK_IS_WAYLAND_DISPLAY is not defined.
>>
>> I was able to get it to compile by changing this particular #ifdef to
>> just be
>>   #ifdef GDK_WINDOWING_WAYLAND
> 
> I think that makes sense. It's defined in gdkconfig.h which is included by gdk.h. As long as it compiles on 3.10, it is good enough. Would you like to send a patch?

Patch sent.

Cheers,

Jeremy

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]