[Mesa-dev] [PATCH 3/4] configure: error out when building xa only with swrast

[Matt Turner]

On Mon, Feb 24, 2014 at 3:05 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> Building to provide accelration using swrast does not make
> sense.
>
> Note: update your build script to explicitly mention svga
> in the gallium drivers list, if you are building the vmwgfx
> xa library.
>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  configure.ac | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 6db3c1d..d98dca9 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1234,6 +1234,9 @@ dnl
>  dnl XA configuration
>  dnl
>  if test "x$enable_xa" = xyes; then
> +    if test "x$gallium_drivers" = xswrast; then
> +       AC_MSG_ERROR([Building xa requires at least one non swrast gallium driver])
> +    fi
>      GALLIUM_STATE_TRACKERS_DIRS="xa $GALLIUM_STATE_TRACKERS_DIRS"
>      enable_gallium_loader=yes
>      enable_gallium_drm_loader=yes
> --
> 1.9.0

It's a never ending battle if we try to catch all kinds of nonsensical
build configurations. I don't think it's worth it, except in a few
cases.

In fact, doesn't xf86-video-vmware just link against libxatracker? I
don't think preventing xatracker from building without a vmwgfx target
is even correct.