AuroraMiddleware/SPIRV-Tools
1
0
Fork 0
mirror of https://github.com/KhronosGroup/SPIRV-Tools synced 2024-11-21 19:20:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Bazel: Use @platforms//os:windows instead of @build_tools//src/conditions:windows (#5042)

Browse Source 
Bazel has been changing its conventions for implementing
conditions on target OS. Update to the latest recommendation.
See https://bazel.build/extending/platforms

Silences a deprecation warning.

Fixes: #5037
This commit is contained in:
David Neto 2022-12-22 08:13:40 -05:00 committed by GitHub
parent 077b09c32a
commit f416d39bb8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
build_defs.bzl
View File

@ -4,7 +4,9 @@ COMMON_COPTS = [
"-DSPIRV_CHECK_CONTEXT",
"-DSPIRV_COLOR_TERMINAL",
] + select({
"@bazel_tools//src/conditions:windows": [],
# On Windows, assume MSVC.
# C++14 is the default in VisualStudio 2017.
"@platforms//os:windows": [],
"//conditions:default": [
"-DSPIRV_LINUX",
"-DSPIRV_TIMER_ENABLED",
@ -27,7 +29,7 @@ COMMON_COPTS = [
TEST_COPTS = COMMON_COPTS + [
] + select({
"@bazel_tools//src/conditions:windows": [
"@platforms//os:windows": [
# Disable C4503 "decorated name length exceeded" warning,
# triggered by some heavily templated types.
# We don't care much about that in test code.