Skip to contentSkip to conversation

package/x11r7/xserver_xorg-server: add --enable-libdrm for udev

PATCH New Gustavo Zacarias 2 participants 2 messages Mar 31, 2015

lore
Share & tools
lore

Gustavo Zacarias

author
#0 lore raw

PATCH 1/1 package/x11r7/xserver_xorg-server: add --enable-libdrm for udev

If the libdrm package is enabled, the system is built with an udev provider and MESA3D_DRI_DRIVER is disabled this adds a --disable-libdrm to configure options, but udev still requires libdrm and results in a build failure since libdrm include paths aren't filled in (missing drm.h include error).
So add --enable-libdrm for the udev scenario to counteract the previous --disable-libdrm.

Signed-off-by Gustavo Zacarias <[email protected]>
---
 package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/x11r7/xserver_xorg-server/xserver_xorg-server.mk +1 -1
index bb5381c..d9f36fc 100644
@@ -130,7 +130,7 @@ 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
+XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev-kms --enable-libdrm
 else
 XSERVER_XORG_SERVER_CONF_OPTS += --disable-config-udev-kms
 endif
Signature · 1 line
2.0.5
Applied
#1 lore raw

PATCH 1/1 package/x11r7/xserver_xorg-server: add --enable-libdrm for udev

Dear Gustavo Zacarias,

Quoted 8 lines · Gustavo Zacarias

If the libdrm package is enabled, the system is built with an udev provider and MESA3D_DRI_DRIVER is disabled this adds a --disable-libdrm to configure options, but udev still requires libdrm and results in a build failure since libdrm include paths aren't filled in (missing drm.h include error).
So add --enable-libdrm for the udev scenario to counteract the previous --disable-libdrm.

Quoted 4 lines
Signed-off-by Gustavo Zacarias <[email protected]>
---
 package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks. You forgot PATCH v2 and a changelog, though :)

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com