Hi Varad,

On 17 November 2016 at 11:56, Varad Gautam <varadgau...@gmail.com> wrote:
> diff --git a/configure.ac b/configure.ac
> index d084d32..b959637 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -392,12 +392,21 @@ AC_ARG_ENABLE(simple-dmabuf-drm-client,
>                               [do not build the simple dmabuf drm client]),,
>                enable_simple_dmabuf_drm_client="auto")
>  if ! test "x$enable_simple_dmabuf_drm_client" = "xno"; then
> -  PKG_CHECK_MODULES(SIMPLE_DMABUF_DRM_CLIENT, [wayland-client libdrm 
> libdrm_intel],
> -                   have_simple_dmabuf_drm_client=yes, 
> have_simple_dmabuf_drm_client=no)
> -  if test "x$have_simple_dmabuf_drm_client" = "xno" -a 
> "x$enable_simple_dmabuf_drm_client" = "xyes"; then
> -    AC_MSG_ERROR([DRM dmabuf client explicitly enabled, but libdrm_intel 
> couldn't be found])
> +  PKG_CHECK_MODULES(SIMPLE_DMABUF_DRM_CLIENT, [wayland-client libdrm],
> +    [PKG_CHECK_MODULES(LIBDRM_PLATFORM, [libdrm_freedreno],
> +      AC_DEFINE([HAVE_LIBDRM_FREEDRENO], [1], [Build freedreno dmabuf 
> client]) have_simple_dmabuf_drm_client=freedreno,
> +    [PKG_CHECK_MODULES(LIBDRM_PLATFORM, [libdrm_intel],
> +      AC_DEFINE([HAVE_LIBDRM_INTEL], [1], [Build intel dmabuf client]) 
> have_simple_dmabuf_drm_client=intel,
> +    have_simple_dmabuf_drm_client=unsupported)])],
> +  have_simple_dmabuf_drm_client=unsupported)
> +
> +  if test "x$have_simple_dmabuf_drm_client" = "xunsupported" -a 
> "x$enable_simple_dmabuf_drm_client" = "xyes"; then
> +    AC_MSG_ERROR([DRM dmabuf client explicitly enabled, but libdrm_intel or 
> libdrm_freedreno not found])
> +  fi
> +
> +  if test "x$have_simple_dmabuf_drm_client" = "xfreedreno" -o 
> "x$have_simple_dmabuf_drm_client" = "xintel"; then
> +    enable_simple_dmabuf_drm_client="yes"
>    fi
> -  enable_simple_dmabuf_drm_client="$have_simple_dmabuf_drm_client"
>  fi
>  AM_CONDITIONAL(BUILD_SIMPLE_DMABUF_DRM_CLIENT, test 
> "x$enable_simple_dmabuf_drm_client" = "xyes")

This conditional ladder pains my eys, but I can't think of anything
better at this point.

Acked-by: Daniel Stone <dani...@collabora.com>

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to