From 6147d675d110793dcb9fcc1b487617450f5c25c0 Mon Sep 17 00:00:00 2001 From: Chinmay Garde Date: Mon, 1 Jul 2019 16:18:20 -0700 Subject: [PATCH] Fix warning about settings unavailable GN arg build_glfw_shell MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This is only available on Darwin & Linux host targets. On Android target builds, the logic amounted to says “Don’t build GLFW on Android targets”. Since GLFW on Android does not make sense and the target does not exist, GN was generating a warning. --- tools/gn | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/gn b/tools/gn index 66413f44fcfe9..b59ac39ba4b9f 100755 --- a/tools/gn +++ b/tools/gn @@ -253,7 +253,8 @@ def to_gn_args(args): if sys.platform == 'darwin': gn_args['mac_sdk_path'] = args.mac_sdk_path - gn_args['build_glfw_shell'] = args.build_glfw_shell + if args.build_glfw_shell: + gn_args['build_glfw_shell'] = True if gn_args['mac_sdk_path'] == '': gn_args['mac_sdk_path'] = os.getenv('FLUTTER_MAC_SDK_PATH', '')