Quoting Emil Velikov (2018-12-13 08:05:50)
> From: Emil Velikov <emil.veli...@collabora.com>
> 
> The gallium drivers do not require a DRI loader. Drop the artificial
> and unnecessary restriction.
> 
> Fixes: af9d276134d ("meson: build libmesa_gallium")
> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
> ---
> There are a few use cases:
> 
>  - building a gallium state-tracker, say opencl in st/clover, without
> having to build the dri one (st/dri)
> Use case: shipping only a opencl/omx/va driver.
> 
>  - building dri loader without an actual DRI driver.
> Use case: working/updating libGL/libGLX_mesa and egl/gbm equivalents.
> 
> This patch should address the former, although the latter seems figgly
> to fix from a quick look. Input and patches will be appreciated.
> ---
>  meson.build | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index db2d27d2459..2aa2794f971 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -387,9 +387,9 @@ endif
>  if with_any_vk and (with_platform_x11 and not with_dri3)
>    error('Vulkan drivers require dri3 for X11 support')
>  endif
> -if with_dri or with_gallium
> -  if with_glx == 'disabled' and not with_egl and not with_platform_haiku
> -    error('building dri or gallium drivers require at least one window 
> system')
> +if with_dri
> +  if with_glx == 'disabled' and not with_egl and not with_gbm
> +    error('building dri drivers require at least one windowing system')
>    endif
>  endif
>  
> -- 
> 2.19.2
> 

I'm not super convinced that building the loader and drivers separately is a
good idea, but building just clover (or media I guess) does seem like a real use
case.

Reviewed-by: Dylan Baker <dy...@pnwbakers.com>

Attachment: signature.asc
Description: signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to