[Mesa-dev] [PATCH] configure.ac: Fail if egl x11 platform dependencies are not available

[Kristian Høgsberg]

On Fri, Jun 1, 2012 at 9:52 PM, Eric Anholt <eric at anholt.net> wrote:
> On Fri,  1 Jun 2012 14:29:44 -0400, Kristian Høgsberg <krh at bitplanet.net> wrote:
>> Currently, if you pass --with-egl-platforms=x11 but xcb-dri2 isn't available
>> we just silently fail and disables building the EGL DRI2 driver.
>>
>> This commit cleans up the EGL platfrom checking and fails if a selected
>> platform can't find its required dependencies.
>
>> +     wayland)
>> +             HAVE_EGL_DRIVER_DRI2=1
>>               PKG_CHECK_MODULES([WAYLAND], [wayland-client wayland-server],, \
>>                                 [AC_MSG_ERROR([cannot find libwayland-client])])
>> -                GALLIUM_WINSYS_DIRS="$GALLIUM_WINSYS_DIRS sw/wayland"
>> +             GALLIUM_WINSYS_DIRS="$GALLIUM_WINSYS_DIRS sw/wayland"
>> +
>> +             m4_ifdef([WAYLAND_SCANNER_RULES],
>> +                      [WAYLAND_SCANNER_RULES(['$(top_srcdir)/src/egl/wayland/wayland-drm/protocol'])])
>> +             ;;
>> +
>
>
> I note that this also turns on HAVE_EGL_DRIVER_DRI2 for a few
> platforms.  That appears to be correct, but I wasn't expecting it from
> the commit mesage.

Ah, you're right, and that's problem if you're only looking to build
the gallium side of egl.  So we need to keep the enable_dri test and
enable HAVE_EGL_DRIVER_DRI2 if that's set, and then the platform
checks don't need to set it.  I'll send out v2.

Kristian