Skip to content

Commit d588c4d

Browse files
committed
bazel: Resolve toolchains correctly
Signed-off-by: Ryan Northey <[email protected]>
1 parent 04b7c78 commit d588c4d

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

.bazelrc

+2-7
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ build:clang --action_env=BAZEL_COMPILER=clang
9090
build:clang --linkopt=-fuse-ld=lld
9191
build:clang --action_env=CC=clang --host_action_env=CC=clang
9292
build:clang --action_env=CXX=clang++ --host_action_env=CXX=clang++
93-
build:clang --incompatible_enable_cc_toolchain_resolution=false
9493

9594
# Flags for Clang + PCH
9695
build:clang-pch --spawn_strategy=local
@@ -116,7 +115,7 @@ build:gcc --cxxopt=-Wno-missing-requires
116115
# not in GCC 11 and GCC 11 is what is used in docker-gcc
117116
# configuration currently
118117
build:gcc --cxxopt=-Wno-unknown-warning
119-
build:gcc --incompatible_enable_cc_toolchain_resolution=false
118+
build:gcc --config=linux
120119

121120
# Clang-tidy
122121
# TODO(phlax): enable this, its throwing some errors as well as finding more issues
@@ -276,19 +275,16 @@ build:cache-local --remote_cache=grpc://localhost:9092
276275

277276
# Remote execution: https://docs.bazel.build/versions/master/remote-execution.html
278277
build:rbe-toolchain --action_env=BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=1
279-
build:rbe-toolchain --incompatible_enable_cc_toolchain_resolution=false
280278

281279
build:rbe-toolchain-clang --config=rbe-toolchain
282280
build:rbe-toolchain-clang --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_clang_platform
283281
build:rbe-toolchain-clang --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_clang_platform
284-
build:rbe-toolchain-clang --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/clang/cc:toolchain
285282
build:rbe-toolchain-clang --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/clang/config:cc-toolchain
286283
build:rbe-toolchain-clang --action_env=CC=clang --action_env=CXX=clang++
287284

288285
build:rbe-toolchain-clang-libc++ --config=rbe-toolchain
289286
build:rbe-toolchain-clang-libc++ --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_clang_libcxx_platform
290287
build:rbe-toolchain-clang-libc++ --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_clang_libcxx_platform
291-
build:rbe-toolchain-clang-libc++ --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/cc:toolchain
292288
build:rbe-toolchain-clang-libc++ --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/config:cc-toolchain
293289
build:rbe-toolchain-clang-libc++ --action_env=CC=clang --action_env=CXX=clang++
294290
build:rbe-toolchain-clang-libc++ --action_env=CXXFLAGS=-stdlib=libc++
@@ -298,7 +294,6 @@ build:rbe-toolchain-clang-libc++ --define force_libcpp=enabled
298294
build:rbe-toolchain-arm64-clang-libc++ --config=rbe-toolchain
299295
build:rbe-toolchain-arm64-clang-libc++ --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_arm64_clang_libcxx_platform
300296
build:rbe-toolchain-arm64-clang-libc++ --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_arm64_clang_libcxx_platform
301-
build:rbe-toolchain-arm64-clang-libc++ --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/cc:toolchain
302297
build:rbe-toolchain-arm64-clang-libc++ --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/clang_libcxx/config:cc-toolchain-arm64
303298
build:rbe-toolchain-arm64-clang-libc++ --action_env=CC=clang --action_env=CXX=clang++
304299
build:rbe-toolchain-arm64-clang-libc++ --action_env=CXXFLAGS=-stdlib=libc++
@@ -322,10 +317,10 @@ build:rbe-toolchain-tsan --linkopt=-L/opt/libcxx_tsan/lib
322317
build:rbe-toolchain-tsan --linkopt=-Wl,-rpath,/opt/libcxx_tsan/lib
323318
build:rbe-toolchain-tsan --config=clang-tsan
324319

320+
build:rbe-toolchain-gcc --config=gcc
325321
build:rbe-toolchain-gcc --config=rbe-toolchain
326322
build:rbe-toolchain-gcc --platforms=@envoy//bazel/rbe/toolchains:rbe_linux_gcc_platform
327323
build:rbe-toolchain-gcc --host_platform=@envoy//bazel/rbe/toolchains:rbe_linux_gcc_platform
328-
build:rbe-toolchain-gcc --crosstool_top=@envoy//bazel/rbe/toolchains/configs/linux/gcc/cc:toolchain
329324
build:rbe-toolchain-gcc --extra_toolchains=@envoy//bazel/rbe/toolchains/configs/linux/gcc/config:cc-toolchain
330325

331326
build:remote --spawn_strategy=remote,sandboxed,local

WORKSPACE

+7
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,10 @@ envoy_dependency_imports()
3131
load("//bazel:dependency_imports_extra.bzl", "envoy_dependency_imports_extra")
3232

3333
envoy_dependency_imports_extra()
34+
35+
register_toolchains(
36+
"//bazel/rbe/toolchains/configs/linux/clang/config:cc-toolchain",
37+
"//bazel/rbe/toolchains/configs/linux/clang_libcxx/config:cc-toolchain",
38+
"//bazel/rbe/toolchains/configs/linux/clang_libcxx/config:cc-toolchain-arm64",
39+
"//bazel/rbe/toolchains/configs/linux/gcc/config:cc-toolchain",
40+
)

bazel/rbe/toolchains/configs/linux/gcc/cc/BUILD

+1-1
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ cc_toolchain_config(
9797
"/usr/include/x86_64-linux-gnu/c++/11",
9898
"/usr/include/c++/11/backward",
9999
],
100-
cxx_flags = ["-std=c++0x"],
100+
cxx_flags = ["-std=c++20"],
101101
dbg_compile_flags = ["-g"],
102102
host_system_name = "local",
103103
link_flags = [

bazel/rbe/toolchains/configs/linux/gcc/config/BUILD

-2
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ toolchain(
2424
exec_compatible_with = [
2525
"@platforms//os:linux",
2626
"@platforms//cpu:x86_64",
27-
"@bazel_tools//tools/cpp:clang",
2827
],
2928
target_compatible_with = [
3029
"@platforms//os:linux",
@@ -39,7 +38,6 @@ platform(
3938
constraint_values = [
4039
"@platforms//os:linux",
4140
"@platforms//cpu:x86_64",
42-
"@bazel_tools//tools/cpp:clang",
4341
],
4442
exec_properties = {
4543
"container-image": "docker://gcr.io/envoy-ci/envoy-build@sha256:56b66cc84065c88a141963cedbbe4198850ffae0dacad769f516d0e9081439da",

0 commit comments

Comments
 (0)