[PATCH 3/3] configure.ac: remove --enable-aiglx option
Jon Turney
jon.turney at dronecode.org.uk
Fri Oct 7 12:58:05 UTC 2016
On 06/10/2016 19:15, Emil Velikov wrote:
> On 6 October 2016 at 19:02, Jon Turney <jon.turney at dronecode.org.uk> wrote:
>> On 29/09/2016 18:41, Emil Velikov wrote:
>>>
>>> Presently the option guards both direct and accelerated indirect GLX. As
>>> such when one toggles it off they end up without any acceleration.
>>>
>>> Remove the option all together until we have the time to split/rework
>>> things.
>>>
>>> Cc: Jon Turney <jon.turney at dronecode.org.uk>
>>> Cc: Adam Jackson <ajax at redhat.com>
>>> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
>>> ---
>>> Jon, I've not checked the Xwin side of things but considering that the
>>> option is enabled by default and having it for Xwin only will be
>>> confusing I've nuked the guards throughout the tree.
>>
>> Sorry I didn't get around to testing this before it was committed.
>>
>> This breaks my build (See [1]), as the DRI2 loader is now built
>> unconditionally, which fails without drm.h
>>
>> I'm not sure exactly what problem this change is fixing, so I'm not sure how
>> to address that.
>>
>> Is it ok to restore the part which makes building the DRI2 loader
>> conditional?
>>
> I had a bad feeling about this, fortunately it seems pretty easy to handle.
>
> From a quick look nothing in glx/glxdri2.c should require libdrm so we
> can nuke the drm.h and xf86drm.h includes, which will get you back up
> and going. Alternatively we can add those in a ifdef WITH_LIBDRM/endif
> block.
That's not quite enough, as building glxdri2.c also requires dri2proto
headers.
At the moment, configure.ac only requires dri2proto when --enable-dri2
turns on.
So either that needs to be made unconditional, or building glxdri2.c
made conditional on DRI2 (untested patch attached)
> Even then we can make the compilation of the DRI2 loader but let's not
> call it AIGLX ;-)
-------------- next part --------------
From db2fcb2636a659abd0998d82204813fd5dcd57ff Mon Sep 17 00:00:00 2001
From: Jon Turney <jon.turney at dronecode.org.uk>
Date: Thu, 6 Oct 2016 22:13:07 +0100
Subject: [PATCH xserver] Don't build DRI loader if DRI2 isn't enabled
---
glx/Makefile.am | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/glx/Makefile.am b/glx/Makefile.am
index fc0b76a..9af7080 100644
--- a/glx/Makefile.am
+++ b/glx/Makefile.am
@@ -16,11 +16,10 @@ AM_CPPFLAGS = \
-I$(top_srcdir)/hw/xfree86/os-support/bus \
-I$(top_srcdir)/hw/xfree86/common \
-I$(top_srcdir)/hw/xfree86/dri \
+ -I$(top_srcdir)/hw/xfree86/dri2
-I$(top_srcdir)/mi \
-I$(top_srcdir)/present
-AM_CPPFLAGS += -I$(top_srcdir)/hw/xfree86/dri2
-
indirect_sources = \
indirect_dispatch.c \
indirect_dispatch.h \
@@ -33,7 +32,9 @@ indirect_sources = \
indirect_table.c
libglxdri_la_SOURCES =
+if DRI2
libglxdri_la_SOURCES += glxdri2.c
+endif
libglxdri_la_LIBADD = $(DLOPEN_LIBS)
--
2.8.3
More information about the xorg-devel
mailing list