On 01/03/2012 12:32 PM, Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > Detect whether a new enough version of XCB is installed at configure > time. If it is not, don't enable the extension and don't build the > unit tests. > > Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> > --- > Using this patch, I was able to successfully build and run both with > and without updated XCB bits installed. I think the lesson learned > here is that it's nonsense for the GLX part of XCB to live in the XCB > library. There's only one user, so it should live there. Seriously. > > configure.ac | 19 +++++++++++++++++++ > src/glx/clientinfo.c | 5 ++++- > src/glx/create_context.c | 2 +- > src/glx/glxextensions.c | 4 ++-- > tests/glx/Makefile.am | 4 +++- > 5 files changed, 29 insertions(+), 5 deletions(-)
Looks okay to me. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
