On Monday, 2018-08-06 17:50:48 -0700, Dylan Baker wrote:
> ---
>  meson.build | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index c7dd5ddfec6..788021c05e9 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1027,9 +1027,9 @@ endif
>  if cc.has_function('dlopen')

This check is not needed on windows, is it? It will always fail afaict.

How about this?
  if host == windows or cc.has_function(dlopen)

>    dep_dl = null_dep
>  else
> -  dep_dl = cc.find_library('dl')
> +  dep_dl = cc.find_library('dl', required : host_machine.system() != 
> 'windows')
>  endif
> -if cc.has_function('dladdr', dependencies : dep_dl)
> +if host_machine.system() != 'windows' and cc.has_function('dladdr', 
> dependencies : dep_dl)
>    # This is really only required for megadrivers
>    pre_args += '-DHAVE_DLADDR'
>  endif

With that, 1-4 and 7-8 are:
Reviewed-by: Eric Engestrom <eric.engest...@intel.com>

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

Reply via email to