Skip to content

Commit

Permalink
[bazel] Remove selects on CPU
Browse files Browse the repository at this point in the history
In a future version of bazel this produces a warning. In this case using
only the platform being windows is enough. Fixes:

```
WARNING: /.../benchmark/BUILD.bazel:29:15: in config_setting rule //:windows: select() on cpu is deprecated. Use platform constraints instead: https://bazel.build/docs/configurable-attributes#platforms.
```
  • Loading branch information
keith committed Dec 20, 2024
1 parent f4f93b5 commit 4ec420b
Showing 1 changed file with 0 additions and 18 deletions.
18 changes: 0 additions & 18 deletions BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -17,30 +17,12 @@ COPTS = [
"-Werror=old-style-cast",
]

config_setting(
name = "qnx",
constraint_values = ["@platforms//os:qnx"],
values = {
"cpu": "x64_qnx",
},
visibility = [":__subpackages__"],
)

config_setting(
name = "windows",
constraint_values = ["@platforms//os:windows"],
values = {
"cpu": "x64_windows",
},
visibility = [":__subpackages__"],
)

config_setting(
name = "macos",
constraint_values = ["@platforms//os:macos"],
visibility = ["//visibility:public"],
)

config_setting(
name = "perfcounters",
define_values = {
Expand Down

0 comments on commit 4ec420b

Please # to comment.