* [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm
2015-07-30 22:59 [Buildroot] [PATCH 0/3 v2] xorg: fix dependencies, add fbturbo driver (branch yem/xorg) Yann E. MORIN
@ 2015-07-30 22:59 ` Yann E. MORIN
2015-07-31 5:57 ` Thomas Petazzoni
2015-07-30 22:59 ` [Buildroot] [PATCH 2/3 v2] package/xserver_xorg-server: dri2 needs libdrm Yann E. MORIN
2015-07-30 22:59 ` [Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package Yann E. MORIN
2 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2015-07-30 22:59 UTC (permalink / raw)
To: buildroot
Currently, we have two locations that may want to enable and.or disable
libdrm, and it's not clear how those two location correlate one to the
other, i.e. they may well want o do one the opposite of the other.
Introduce an intermediate variable that records whether libdrm is
needed, and use that to depend on, and enable, libdrm.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
---
Note: there might be other libs in the same situation, but that can come
in later patches as the need arises... I just change this one lbdrm,
because it will be needed by a followup patch to fix building with
dri2proto enabled.
---
.../x11r7/xserver_xorg-server/xserver_xorg-server.mk | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
index 79d663c..374a9c3 100644
--- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
+++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
@@ -122,10 +122,11 @@ XSERVER_XORG_SERVER_CONF_OPTS += --disable-kdrive --disable-xfbdev
endif
ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
-XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-libdrm --enable-glx
-XSERVER_XORG_SERVER_DEPENDENCIES += libdrm mesa3d xproto_xf86driproto
+XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
+XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d xproto_xf86driproto
+XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
else
-XSERVER_XORG_SERVER_CONF_OPTS += --disable-dri --disable-libdrm --disable-glx
+XSERVER_XORG_SERVER_CONF_OPTS += --disable-dri --disable-glx
endif
ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_AIGLX),y)
@@ -145,8 +146,8 @@ XSERVER_XORG_SERVER_DEPENDENCIES += udev
XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev
# udev kms support depends on libdrm
ifeq ($(BR2_PACKAGE_LIBDRM),y)
-XSERVER_XORG_SERVER_DEPENDENCIES += libdrm
-XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev-kms --enable-libdrm
+XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev-kms
+XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
else
XSERVER_XORG_SERVER_CONF_OPTS += --disable-config-udev-kms
endif
@@ -228,4 +229,11 @@ XSERVER_XORG_SERVER_CONF_OPTS += --with-sha1=libsha1
XSERVER_XORG_SERVER_DEPENDENCIES += libsha1
endif
+ifeq ($(XSERVER_XORG_SERVER_NEEDS_LIBDRM),y)
+XSERVER_XORG_SERVER_DEPENDENCIES += libdrm
+XSERVER_XORG_SERVER_CONF_OPTS += --enable-libdrm
+else
+XSERVER_XORG_SERVER_CONF_OPTS += --disable-libdrm
+endif
+
$(eval $(autotools-package))
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm
2015-07-30 22:59 ` [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm Yann E. MORIN
@ 2015-07-31 5:57 ` Thomas Petazzoni
2015-07-31 16:59 ` Yann E. MORIN
0 siblings, 1 reply; 9+ messages in thread
From: Thomas Petazzoni @ 2015-07-31 5:57 UTC (permalink / raw)
To: buildroot
Yann,
On Fri, 31 Jul 2015 00:59:05 +0200, Yann E. MORIN wrote:
> ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
> -XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-libdrm --enable-glx
> -XSERVER_XORG_SERVER_DEPENDENCIES += libdrm mesa3d xproto_xf86driproto
> +XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
> +XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d xproto_xf86driproto
> +XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
Hum, didn't we say that this should be a hidden Config.in boolean?
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm
2015-07-31 5:57 ` Thomas Petazzoni
@ 2015-07-31 16:59 ` Yann E. MORIN
2015-08-05 20:11 ` Yann E. MORIN
0 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2015-07-31 16:59 UTC (permalink / raw)
To: buildroot
Thomas, All,
On 2015-07-31 07:57 +0200, Thomas Petazzoni spake thusly:
> On Fri, 31 Jul 2015 00:59:05 +0200, Yann E. MORIN wrote:
> > ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
> > -XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-libdrm --enable-glx
> > -XSERVER_XORG_SERVER_DEPENDENCIES += libdrm mesa3d xproto_xf86driproto
> > +XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
> > +XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d xproto_xf86driproto
> > +XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
>
> Hum, didn't we say that this should be a hidden Config.in boolean?
Yes, you suggested so; I said yes; I tried; I said no... ;-)
http://lists.busybox.net/pipermail/buildroot/2015-July/135100.html
Basically, all the conditional code is in the .mk , and there's almost
no conditional in the Config.in.
So, I'd like not to duplicate the decision making in Config.in and keep
all in a single place, the .mk .
That's why I did not move that to a blind kconfig option.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm
2015-07-31 16:59 ` Yann E. MORIN
@ 2015-08-05 20:11 ` Yann E. MORIN
2015-08-08 12:04 ` Bernd Kuhls
0 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2015-08-05 20:11 UTC (permalink / raw)
To: buildroot
Thomas, All,
On 2015-07-31 18:59 +0200, Yann E. MORIN spake thusly:
> On 2015-07-31 07:57 +0200, Thomas Petazzoni spake thusly:
> > On Fri, 31 Jul 2015 00:59:05 +0200, Yann E. MORIN wrote:
> > > ifeq ($(BR2_PACKAGE_MESA3D_DRI_DRIVER),y)
> > > -XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-libdrm --enable-glx
> > > -XSERVER_XORG_SERVER_DEPENDENCIES += libdrm mesa3d xproto_xf86driproto
> > > +XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
> > > +XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d xproto_xf86driproto
> > > +XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
> >
> > Hum, didn't we say that this should be a hidden Config.in boolean?
>
> Yes, you suggested so; I said yes; I tried; I said no... ;-)
> http://lists.busybox.net/pipermail/buildroot/2015-July/135100.html
>
> Basically, all the conditional code is in the .mk , and there's almost
> no conditional in the Config.in.
>
> So, I'd like not to duplicate the decision making in Config.in and keep
> all in a single place, the .mk .
>
> That's why I did not move that to a blind kconfig option.
Did you make up your mind about this?
Now that we've switched to a mdular server by default, this series is
now fixing actual autobuild errors...
If you really prefer that we duplicate all the logic in Config.in , I'll
do it.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 2/3 v2] package/xserver_xorg-server: dri2 needs libdrm
2015-07-30 22:59 [Buildroot] [PATCH 0/3 v2] xorg: fix dependencies, add fbturbo driver (branch yem/xorg) Yann E. MORIN
2015-07-30 22:59 ` [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm Yann E. MORIN
@ 2015-07-30 22:59 ` Yann E. MORIN
2015-07-30 22:59 ` [Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package Yann E. MORIN
2 siblings, 0 replies; 9+ messages in thread
From: Yann E. MORIN @ 2015-07-30 22:59 UTC (permalink / raw)
To: buildroot
Fixes build issues like:
CC dri2.lo
CC dri2ext.lo
dri2.c: In function 'dri2_probe_driver_name':
dri2.c:1434:9: error: unknown type name 'drmVersionPtr'
dri2.c:1434:9: warning: implicit declaration of function 'drmGetVersion' [-Wimplicit-function-declaration]
dri2.c:1434:9: warning: nested extern declaration of 'drmGetVersion' [-Wnested-externs]
dri2.c:1444:40: error: invalid type argument of '->' (have 'int')
dri2.c:1444:55: error: invalid type argument of '->' (have 'int')
dri2.c:1445:9: warning: implicit declaration of function 'drmFreeVersion' [-Wimplicit-function-declaration]
dri2.c:1445:9: warning: nested extern declaration of 'drmFreeVersion' [-Wnested-externs]
make[7]: *** [dri2.lo] Error 1
with this defconfig:
BR2_TOOLCHAIN_EXTERNAL=y
BR2_PACKAGE_XORG7=y
BR2_PACKAGE_XSERVER_XORG_SERVER=y
BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR=y
BR2_PACKAGE_XPROTO_DRI2PROTO=y
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/x11r7/xserver_xorg-server/Config.in | 1 +
package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 1 +
2 files changed, 2 insertions(+)
diff --git a/package/x11r7/xserver_xorg-server/Config.in b/package/x11r7/xserver_xorg-server/Config.in
index f0ec598..5a6814b 100644
--- a/package/x11r7/xserver_xorg-server/Config.in
+++ b/package/x11r7/xserver_xorg-server/Config.in
@@ -93,6 +93,7 @@ choice
config BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR
bool "Modular X.org"
depends on BR2_INSTALL_LIBSTDCPP
+ select BR2_PACKAGE_LIBDRM if BR2_PACKAGE_XPROTO_DRI2PROTO
help
This variant of the X.org server is the full-blown variant,
as used by desktop GNU/Linux distributions. The drivers (for
diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
index 374a9c3..7115e3f 100644
--- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
+++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
@@ -190,6 +190,7 @@ ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR),y)
ifeq ($(BR2_PACKAGE_XPROTO_DRI2PROTO),y)
XSERVER_XORG_SERVER_DEPENDENCIES += xproto_dri2proto
XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri2
+XSERVER_XORG_SERVER_NEEDS_LIBDRM = y
else
XSERVER_XORG_SERVER_CONF_OPTS += --disable-dri2
endif
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package
2015-07-30 22:59 [Buildroot] [PATCH 0/3 v2] xorg: fix dependencies, add fbturbo driver (branch yem/xorg) Yann E. MORIN
2015-07-30 22:59 ` [Buildroot] [PATCH 1/3 v2] package/xserver: commonalise dependency on libdrm Yann E. MORIN
2015-07-30 22:59 ` [Buildroot] [PATCH 2/3 v2] package/xserver_xorg-server: dri2 needs libdrm Yann E. MORIN
@ 2015-07-30 22:59 ` Yann E. MORIN
2015-10-03 17:37 ` Peter Korsgaard
2 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2015-07-30 22:59 UTC (permalink / raw)
To: buildroot
From: Scott Fan <fancp2007@gmail.com>
Add the fbturbo video driver, which is based on xf86-video-fbdev (with
none of the original features stripped), primarily optimized for the
devices powered by the Allwinner SoC (A10, A13, A20).
https://github.com/ssvb/xf86-video-fbturbo/
Signed-off-by: Scott Fan <fancp2007@gmail.com>
[yann.morin.1998 at free.fr: strip help text to the basics only]
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
---
package/x11r7/Config.in | 1 +
package/x11r7/xdriver_xf86-video-fbturbo/Config.in | 16 ++++++++++++++++
.../xdriver_xf86-video-fbturbo.mk | 21 +++++++++++++++++++++
3 files changed, 38 insertions(+)
create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/Config.in
create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
diff --git a/package/x11r7/Config.in b/package/x11r7/Config.in
index 859e895..4e996fc 100644
--- a/package/x11r7/Config.in
+++ b/package/x11r7/Config.in
@@ -162,6 +162,7 @@ if BR2_PACKAGE_XORG7
source package/x11r7/xdriver_xf86-video-cirrus/Config.in
source package/x11r7/xdriver_xf86-video-dummy/Config.in
source package/x11r7/xdriver_xf86-video-fbdev/Config.in
+ source package/x11r7/xdriver_xf86-video-fbturbo/Config.in
source package/x11r7/xdriver_xf86-video-geode/Config.in
source package/x11r7/xdriver_xf86-video-glide/Config.in
source package/x11r7/xdriver_xf86-video-glint/Config.in
diff --git a/package/x11r7/xdriver_xf86-video-fbturbo/Config.in b/package/x11r7/xdriver_xf86-video-fbturbo/Config.in
new file mode 100644
index 0000000..cab12d2
--- /dev/null
+++ b/package/x11r7/xdriver_xf86-video-fbturbo/Config.in
@@ -0,0 +1,16 @@
+config BR2_PACKAGE_XDRIVER_XF86_VIDEO_FBTURBO
+ bool "xf86-video-fbturbo"
+ select BR2_PACKAGE_LIBDRI2
+ select BR2_PACKAGE_XPROTO_FONTSPROTO
+ select BR2_PACKAGE_XPROTO_RANDRPROTO
+ select BR2_PACKAGE_XPROTO_RENDERPROTO
+ select BR2_PACKAGE_XPROTO_VIDEOPROTO
+ select BR2_PACKAGE_XPROTO_XPROTO
+ select BR2_PACKAGE_XPROTO_XF86DRIPROTO
+ help
+ video driver, primarily optimized for the devices powered
+ by the Allwinner SoC (A10, A13, A20). It can use some of
+ the 2D/3D hardware acceleration features.
+
+ https://github.com/ssvb/xf86-video-fbturbo
+
diff --git a/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
new file mode 100644
index 0000000..18f80d5
--- /dev/null
+++ b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
@@ -0,0 +1,21 @@
+################################################################################
+#
+# xdriver_xf86-video-fbturbo
+#
+################################################################################
+
+XDRIVER_XF86_VIDEO_FBTURBO_VERSION = 0.4.0
+XDRIVER_XF86_VIDEO_FBTURBO_SITE = $(call github,ssvb,xf86-video-fbturbo,$(XDRIVER_XF86_VIDEO_FBTURBO_VERSION))
+XDRIVER_XF86_VIDEO_FBTURBO_LICENSE = MIT
+XDRIVER_XF86_VIDEO_FBTURBO_LICENSE_FILES = COPYING
+XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES = xserver_xorg-server xproto_fontsproto xproto_randrproto xproto_renderproto xproto_videoproto xproto_xproto
+
+XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES += xproto_xf86driproto libdri2
+
+define XDRIVER_XF86_VIDEO_FBTURBO_INSTALL_CONF_FILE
+ $(INSTALL) -m 0644 -D $(@D)/xorg.conf $(TARGET_DIR)/etc/X11/xorg.conf
+endef
+
+XDRIVER_XF86_VIDEO_FBTURBO_POST_INSTALL_TARGET_HOOKS += XDRIVER_XF86_VIDEO_FBTURBO_INSTALL_CONF_FILE
+
+$(eval $(autotools-package))
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package
2015-07-30 22:59 ` [Buildroot] [PATCH 3/3 v2] xdriver_xf86-video-fbturbo: new package Yann E. MORIN
@ 2015-10-03 17:37 ` Peter Korsgaard
0 siblings, 0 replies; 9+ messages in thread
From: Peter Korsgaard @ 2015-10-03 17:37 UTC (permalink / raw)
To: buildroot
On Fri, Jul 31, 2015 at 12:59 AM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> From: Scott Fan <fancp2007@gmail.com>
>
> Add the fbturbo video driver, which is based on xf86-video-fbdev (with
> none of the original features stripped), primarily optimized for the
> devices powered by the Allwinner SoC (A10, A13, A20).
>
> https://github.com/ssvb/xf86-video-fbturbo/
>
> Signed-off-by: Scott Fan <fancp2007@gmail.com>
> [yann.morin.1998 at free.fr: strip help text to the basics only]
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> ---
> package/x11r7/Config.in | 1 +
> package/x11r7/xdriver_xf86-video-fbturbo/Config.in | 16 ++++++++++++++++
> .../xdriver_xf86-video-fbturbo.mk | 21 +++++++++++++++++++++
> 3 files changed, 38 insertions(+)
> create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/Config.in
> create mode 100644 package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
>
> +++ b/package/x11r7/xdriver_xf86-video-fbturbo/Config.in
> @@ -0,0 +1,16 @@
> +config BR2_PACKAGE_XDRIVER_XF86_VIDEO_FBTURBO
> + bool "xf86-video-fbturbo"
> + select BR2_PACKAGE_LIBDRI2
It actually doesn't use libdri2 for anything (it does contain a stray
#include "dri2.h" but it isn't needed and I've added a patch to drop
it).
The configure script also checks for libdrm and pixman, so I've added
that as well.
> + select BR2_PACKAGE_XPROTO_FONTSPROTO
> + select BR2_PACKAGE_XPROTO_RANDRPROTO
> + select BR2_PACKAGE_XPROTO_RENDERPROTO
> + select BR2_PACKAGE_XPROTO_VIDEOPROTO
> + select BR2_PACKAGE_XPROTO_XPROTO
> + select BR2_PACKAGE_XPROTO_XF86DRIPROTO
> + help
> + video driver, primarily optimized for the devices powered
> + by the Allwinner SoC (A10, A13, A20). It can use some of
> + the 2D/3D hardware acceleration features.
> +
> + https://github.com/ssvb/xf86-video-fbturbo
> +
> diff --git a/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
> new file mode 100644
> index 0000000..18f80d5
> --- /dev/null
> +++ b/package/x11r7/xdriver_xf86-video-fbturbo/xdriver_xf86-video-fbturbo.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# xdriver_xf86-video-fbturbo
> +#
> +################################################################################
> +
> +XDRIVER_XF86_VIDEO_FBTURBO_VERSION = 0.4.0
> +XDRIVER_XF86_VIDEO_FBTURBO_SITE = $(call github,ssvb,xf86-video-fbturbo,$(XDRIVER_XF86_VIDEO_FBTURBO_VERSION))
> +XDRIVER_XF86_VIDEO_FBTURBO_LICENSE = MIT
> +XDRIVER_XF86_VIDEO_FBTURBO_LICENSE_FILES = COPYING
> +XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES = xserver_xorg-server xproto_fontsproto xproto_randrproto xproto_renderproto xproto_videoproto xproto_xproto
> +
> +XDRIVER_XF86_VIDEO_FBTURBO_DEPENDENCIES += xproto_xf86driproto libdri2
There's no need for the += here, it can all be handled on a single
_DEPENDENCIES assignment. I've fixed the overlong line as well.
The package also has optional support for libpciaccess, so I've added that.
Committed with these fixes, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 9+ messages in thread