On Friday, 7 October 2016, Jon Turney <<a href="mailto:jon.turney@dronecode.org.uk">jon.turney@dronecode.org.uk</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 06/10/2016 19:15, Emil Velikov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 6 October 2016 at 19:02, Jon Turney <<a>jon.turney@dronecode.org.uk</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 29/09/2016 18:41, Emil Velikov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Presently the option guards both direct and accelerated indirect GLX. As<br>
such when one toggles it off they end up without any acceleration.<br>
<br>
Remove the option all together until we have the time to split/rework<br>
things.<br>
<br>
Cc: Jon Turney <<a>jon.turney@dronecode.org.uk</a>><br>
Cc: Adam Jackson <<a>ajax@redhat.com</a>><br>
Signed-off-by: Emil Velikov <<a>emil.l.velikov@gmail.com</a>><br>
---<br>
Jon, I've not checked the Xwin side of things but considering that the<br>
option is enabled by default and having it for Xwin only will be<br>
confusing I've nuked the guards throughout the tree.<br>
</blockquote>
<br>
Sorry I didn't get around to testing this before it was committed.<br>
<br>
This breaks my build (See [1]), as the DRI2 loader is now built<br>
unconditionally, which fails without drm.h<br>
<br>
I'm not sure exactly what problem this change is fixing, so I'm not sure how<br>
to address that.<br>
<br>
Is it ok to restore the part which makes building the DRI2 loader<br>
conditional?<br>
<br>
</blockquote>
I had a bad feeling about this, fortunately it seems pretty easy to handle.<br>
<br>
>From a quick look nothing in glx/glxdri2.c should require libdrm so we<br>
can nuke the drm.h and xf86drm.h includes, which will get you back up<br>
and going. Alternatively we can add those in a ifdef WITH_LIBDRM/endif<br>
block.<br>
</blockquote>
<br>
That's not quite enough, as building glxdri2.c also requires dri2proto headers.<br>
<br>
At the moment, <a href="http://configure.ac" target="_blank">configure.ac</a> only requires dri2proto when --enable-dri2 turns on.<br>
<br>
So either that needs to be made unconditional, or building glxdri2.c made conditional on DRI2 (untested patch attached)<br>
<br></blockquote><div>You're correct. Wrapping it in DRI2 conditional is a good idea.</div><div><br></div><div>Note creating an empty (no sources or static libs) library is likely to cause problems. Just use the form prior to my patch ?</div><div><br></div><div>Emil</div>