diff mbox

[1/1] package/x11r7/xdriver_xf86-video-savage: disable DRI support when not available

Message ID 1470583546-7332-1-git-send-email-bernd.kuhls@t-online.de
State Accepted
Headers show

Commit Message

Bernd Kuhls Aug. 7, 2016, 3:25 p.m. UTC
Fixes
http://autobuild.buildroot.net/results/ecd/ecdb6977d282c276cbdf1f3dd0ccfaddff1d1ac5/

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/x11r7/xdriver_xf86-video-savage/xdriver_xf86-video-savage.mk | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Thomas Petazzoni Aug. 7, 2016, 8:51 p.m. UTC | #1
Hello,

On Sun,  7 Aug 2016 17:25:46 +0200, Bernd Kuhls wrote:

> +ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),)
> +XDRIVER_XF86_VIDEO_SAVAGE_CONF_OPTS = --disable-dri
> +endif

I've applied because we're doing the same condition in other X.org
video drivers. But I'm not happy at all with this condition. Indeed the
build failure is that DRI support is not provided by the X.org server
(for example, the DRIInfoPtr type that is missing and causes the
error), is provided by the X.org server.

So I really believe this whole DRI stuff needs to be reworked.

Thanks,

Thomas
diff mbox

Patch

diff --git a/package/x11r7/xdriver_xf86-video-savage/xdriver_xf86-video-savage.mk b/package/x11r7/xdriver_xf86-video-savage/xdriver_xf86-video-savage.mk
index dc5644a..95bca7e 100644
--- a/package/x11r7/xdriver_xf86-video-savage/xdriver_xf86-video-savage.mk
+++ b/package/x11r7/xdriver_xf86-video-savage/xdriver_xf86-video-savage.mk
@@ -12,4 +12,8 @@  XDRIVER_XF86_VIDEO_SAVAGE_LICENSE_FILES = COPYING
 XDRIVER_XF86_VIDEO_SAVAGE_AUTORECONF = YES
 XDRIVER_XF86_VIDEO_SAVAGE_DEPENDENCIES = xserver_xorg-server libdrm xproto_fontsproto xproto_randrproto xproto_renderproto xproto_videoproto xproto_xextproto xproto_xf86driproto xproto_xproto
 
+ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),)
+XDRIVER_XF86_VIDEO_SAVAGE_CONF_OPTS = --disable-dri
+endif
+
 $(eval $(autotools-package))