Quoting Eric Engestrom (2018-08-23 10:13:17)
> On Wednesday, 2018-08-22 10:04:35 -0700, Dylan Baker wrote:
> > Signed-off-by: Dylan Baker <dylan.c.ba...@intel.com>
> > Reviewed-by: Eric Anholt <e...@anholt.net>
> > ---
> >  meson.build | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> > 
> > diff --git a/meson.build b/meson.build
> > index 1af610573d5..5102ffe0c7c 100644
> > --- a/meson.build
> > +++ b/meson.build
> > @@ -264,8 +264,12 @@ if with_glx == 'auto'
> >    elif with_platform_haiku
> >      with_glx = 'disabled'
> 
> How about simply adding it here, before the with_gallium check?
> 
>   +  elif host_machine.system() == 'windows'
>   +    with_glx = 'disabled'
> 
> Otherwise this opens the door to a weird `(gallium + x11 + gl - vk) on
> windows` bug here.
> 
> With that:
> Reviewed-by: Eric Engestrom <eric.engest...@intel.com>
> 
> >    elif with_gallium
> > -    # Even when building just gallium drivers the user probably wants dri
> > -    with_glx = 'dri'
> > +    if host_machine.system() == 'windows'
> > +      with_glx = 'disabled'
> > +    else
> > +      # Even when building just gallium drivers the user probably wants dri
> > +      with_glx = 'dri'
> > +    endif
> >    elif with_platform_x11 and with_any_opengl and not with_any_vk
> >      # The automatic behavior should not be to turn on xlib based glx when
> >      # building only vulkan drivers
> > -- 
> > 2.18.0
> > 
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev

ping

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