[Mesa-stable] [PATCH] configure.ac: check for libdrm when using VL with X11

Alyssa Rosenzweig alyssa at rosenzweig.io
Mon May 6 23:38:20 UTC 2019


Wrong Alyssa, cc'ing the right one :)

On Mon, May 06, 2019 at 04:32:38PM +0100, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> The X11 specific code uses libdrm, yet we are missing the dependency.
> This has gone unnoticed since all drivers which use VL already mandate
> the library.
> 
> Note: this is applicable only for the stable branches.
> 
> Cc: Alyssa Rosenzweig <alyssa at rosenzweig.io>
> Cc: <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Alyssa this should resolve the failure with minimal churn. Please let
> me know if it works on your end or not.
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 1ef68fe68e6..b288ecbd265 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2357,7 +2357,7 @@ if test "x$enable_xvmc" = xyes -o \
>          "x$enable_omx_tizonia" = xyes -o \
>          "x$enable_va" = xyes; then
>      if echo $platforms | grep -q "x11"; then
> -        PKG_CHECK_MODULES([VL], [x11-xcb xcb xcb-dri2 >= $XCBDRI2_REQUIRED])
> +        PKG_CHECK_MODULES([VL], [x11-xcb xcb xcb-dri2 >= $XCBDRI2_REQUIRED libdrm >= $LIBDRM_REQUIRED])
>      fi
>      need_gallium_vl_winsys=yes
>  fi
> -- 
> 2.21.0


More information about the mesa-stable mailing list