From b30ddee2aa9ce1654986dfb0ff6d0e664ab6f59e Mon Sep 17 00:00:00 2001 From: scentini Date: Tue, 28 Aug 2018 14:07:55 +0200 Subject: use flag_values = {"@bazel_tools//tools/cpp:compiler": "x"} for compiler config_settings --- BUILD | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/BUILD b/BUILD index 25c41a42..58d7a82b 100644 --- a/BUILD +++ b/BUILD @@ -52,10 +52,9 @@ COPTS = select({ ], }) -config_setting( - name = "msvc", - values = { "compiler": "msvc-cl" }, -) +load(":compiler_config_setting.bzl", "create_compiler_config_setting") + +create_compiler_config_setting(name = "msvc", value = "msvc-cl") config_setting( name = "android", -- cgit v1.2.3 From 664f9277b52dd7a68fa4769dc1df4ea9e061267a Mon Sep 17 00:00:00 2001 From: scentini Date: Tue, 28 Aug 2018 14:10:46 +0200 Subject: Create compiler_config_setting.bzl --- compiler_config_setting.bzl | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) create mode 100644 compiler_config_setting.bzl diff --git a/compiler_config_setting.bzl b/compiler_config_setting.bzl new file mode 100644 index 00000000..5e52a652 --- /dev/null +++ b/compiler_config_setting.bzl @@ -0,0 +1,21 @@ +"""Creates config_setting that allows selecting based on 'compiler' value.""" + +def create_compiler_config_setting(name, value): + # The "do_not_use_tools_cpp_compiler_present" attribute exists to + # distinguish between older versions of Bazel that do not support + # "@bazel_tools//tools/cpp:compiler" flag_value, and newer ones that do. + # In the future, the only way to select on the compiler will be through + # flag_values{"@bazel_tools//tools/cpp:compiler"} and the else branch can + # be removed. + if hasattr(cc_common, "do_not_use_tools_cpp_compiler_present"): + native.config_setting( + name = name, + flag_values = { + "@bazel_tools//tools/cpp:compiler": value, + }, + ) + else: + native.config_setting( + name = name, + values = {"compiler": value}, + ) -- cgit v1.2.3