Looks alright to me.
Reviewed-by: Roland Scheidegger <srol...@vmware.com>

Am 25.10.19 um 23:12 schrieb Jose Fonseca:
> - Use parsed options instead of using ARGUMENTS directly.
> - Handle case mingw cross compilation.
> ---
>  SConstruct | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/SConstruct b/SConstruct
> index 61a915f7deb..f905189dd9e 100644
> --- a/SConstruct
> +++ b/SConstruct
> @@ -71,9 +71,8 @@ Help(opts.GenerateHelpText(env))
>  #######################################################################
>  # Print a deprecation warning for using scons on non-windows
>  
> -if common.host_platform != 'windows':
> -    force = ARGUMENTS['force_scons']
> -    if force.lower() not in {'false', 'off', 'none', '0', 'n'}:
> +if common.host_platform != 'windows' and env['platform'] != 'windows':
> +    if env['force_scons']:
>          print("WARNING: Scons is deprecated for non-windows platforms 
> (including cygwin) "
>                "please use meson instead.", file=sys.stderr)
>      else:
> 

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

Reply via email to