[Mesa-dev] [PATCH 1/9] configure.ac: radeonsi requires EGL_DRM and GBM

[Emil Velikov]

On 26/04/14 14:27, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
> 
> ---
>  configure.ac | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/configure.ac b/configure.ac
> index c71fa26..b2ea8ca 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1738,6 +1738,19 @@ gallium_require_drm_loader() {
>      fi
>  }
>  
> +require_egl_drm() {
> +    case "$with_egl_platforms" in
> +        *drm*)
> +            ;;
> +         *)
> +            AC_MSG_ERROR([--with-egl-platforms=drm is required to build the $1 driver.])
> +            ;;
> +    esac
> +    if test "x$enable_gbm" != xyes; then
> +            AC_MSG_ERROR([--enable-gbm is required to build the $1 driver.])
> +    fi
> +}
> +
Hi Marek,

I'm not sure if the patch does exactly what you expect it to.

AFAICS, based on the above you require the drm egl platform with radeonsi,
although it will break the build for people that do not build gbm/egl.

Do I take it that radeonsi is available/usable _only_ when gbm + drm egl is
build ? Otherwise just wrap the above in a

if test "x$enable_egl" = xyes; then
....
fi

Cheers,
-Emil