<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 17, 2016 at 8:00 AM, Derek Foreman <span dir="ltr"><<a href="mailto:derekf@osg.samsung.com" target="_blank">derekf@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 16/02/16 10:37 AM, Daniel Stone wrote:<br>
> Hi,<br>
><br>
> On 16 February 2016 at 16:34, Derek Foreman <<a href="mailto:derekf@osg.samsung.com">derekf@osg.samsung.com</a>> wrote:<br>
>> +try_damage_buffer(struct dri2_egl_surface *dri2_surf,<br>
>> +                  const EGLint *rects,<br>
>> +                  EGLint n_rects)<br>
>> +{<br>
>> +/* The WL_SURFACE_DAMAGE_BUFFER_SINCE_VERSION macro and<br>
>> + * wl_proxy_get_version() were both introduced in wayland 1.10.<br>
>> + * Instead of bumping our wayland dependency we just make this<br>
>> + * function conditional on the required 1.10 features, falling<br>
>> + * back to old (correct but suboptimal) behaviour for older<br>
>> + * wayland.<br>
>> + */<br>
>> +#ifdef WL_SURFACE_DAMAGE_BUFFER_SINCE_VERSION<br>
><br>
> It still bumps the runtime requirement, i.e. once built against >=1.10<br>
> it can only ever be run against >= 1.10. Maybe dlsym is overkill, but<br>
> OTOH maybe not ...<br>
<br>
</span>Yup, that's true.<br>
<br>
I kind of just assumed distros would set their own dependency<br>
information to whatever libwayland they actually built against.<br>
<br>
I hadn't given much thought to building mesa against a new libwayland<br>
then downgrading libwayland or transplanting that mesa to a system with<br>
older wayland.<br>
<br>
I can re-do this with runtime dlsym checking for wl_proxy_get_version()<br>
if that's preferred - someone else can make that decision because I<br>
don't have a strong opinion either way. :)<br></blockquote><div><br></div><div>I just added ajax to the CC.  He's the person who will have to deal with the back-porting fall-out so I'll let him venture an opinion.  Personally, I don't really care either way.<br></div><div>--Jason <br></div></div><br></div></div>