Oops! I didn't know that, sorry for the errant patch then. I'll leave that to someone who knows more about the pieces involved. (I'm happy to hear that though, as it had seemed a shame to make that decision at build time instead of run time.) One thing, though -- why, then, are there any conditionals based on e.g. RENDER_MAJOR? Damian <div class="gmail_quote">On Wed, Aug 12, 2009 at 2:23 AM, Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> wrote: <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im"> </div>We have a private copy of Xrender.h in cairo-xlib-xrender-private.h that is meant to hide the details of what is absent from the main body of the code. I'd prefer a solution that kept the ifdefs out of cairo-xlib-surface.c, so I'd imagine that it'll take a bit of work to conditionally consume a few more Xrender stubs and ensure that buggy_gradients is defined if the headers are too old as well. Thanks, -ickle </blockquote></div>