[Buildroot] [PATCH] package/x11r7/xserver_xorg-server: add --enable-libdrm for udev

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Tue Mar 31 15:47:02 UTC 2015


Dear Gustavo Zacarias,

On Tue, 31 Mar 2015 12:24:53 -0300, Gustavo Zacarias wrote:
> 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 <gustavo at zacarias.com.ar>
> ---
>  package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> index bb5381c..30b7b38 100644
> --- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> +++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
> @@ -126,7 +126,7 @@ endif
>  
>  ifeq ($(BR2_PACKAGE_HAS_UDEV),y)
>  XSERVER_XORG_SERVER_DEPENDENCIES += udev
> -XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev
> +XSERVER_XORG_SERVER_CONF_OPTS += --enable-config-udev --enable-libdrm

Shouldn't this...

>  # udev kms support depends on libdrm
>  ifeq ($(BR2_PACKAGE_LIBDRM),y)
>  XSERVER_XORG_SERVER_DEPENDENCIES += libdrm

be in this if condition, to make sure libdrm is indeed available?

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



More information about the buildroot mailing list