On 12 November 2017 at 07:15, sguttula <[email protected]> wrote:
> In case of chromiumos, minigbm is used in place of mesa-gbm.
> So to make runtime dependency works fine ,ignoring this
> compile time dependency check on mesa-gbm
>
> Cc: [email protected]
> Cc: Emil Velikov <[email protected]>
> Cc: Deepak Sharma <[email protected]>
>
> Signed-off-by: sguttula <[email protected]>
> Reviewed-by: Deepak Sharma <[email protected]>
> ---
>  configure.ac | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>  mode change 100644 => 100755 configure.ac
>
> diff --git a/configure.ac b/configure.ac
> old mode 100644
> new mode 100755
> index bbabf3b..292217c
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1735,7 +1735,7 @@ for plat in $platforms; do
>
>      drm)
>          test "x$enable_gbm" = "xno" &&
> -                AC_MSG_ERROR([EGL platform drm needs gbm])
> +                AC_MSG_WARN([EGL platform drm needs gbm])
Since it's no longer required, I'd reword the commit message a bit.
Say "EGL platform drm requires gbm implementation at runtime"?

The patch should be the final in the series, otherwise we'll get an
interment breakage.

Thanks
Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to