Skip to content

Commit 8171744

Browse files
committed
Switch BCR presubmit to testing from example module (instead of default dummy module) so we can use .bazelrc and reference @local_config_cc.
PiperOrigin-RevId: 727029136
1 parent c07e854 commit 8171744

File tree

1 file changed

+13
-59
lines changed

1 file changed

+13
-59
lines changed

.bcr/presubmit.yml

+13-59
Original file line numberDiff line numberDiff line change
@@ -1,66 +1,20 @@
1-
matrix:
2-
platform: ["debian10", "macos", "ubuntu2004"]
3-
bazel: [7.x]
4-
5-
tasks:
6-
verify_targets:
7-
name: "Verify build targets"
8-
platform: ${{ platform }}
9-
bazel: ${{ bazel }}
10-
build_flags:
11-
- '--host_cxxopt=-std=c++17'
12-
- '--cxxopt=-std=c++17'
13-
build_targets:
14-
- '@protobuf//:protobuf'
15-
- '@protobuf//:protobuf_lite'
16-
- '@protobuf//:protobuf_python'
17-
- '@protobuf//:protobuf_java'
18-
- '@protobuf//:protoc'
19-
- '@protobuf//:test_messages_proto2_cc_proto'
20-
- '@protobuf//:test_messages_proto3_cc_proto'
21-
# Separate windows for different c++ build flags.
22-
verify_targets_windows:
23-
name: "Verify windows build targets"
24-
platform: windows
25-
bazel: ${{ bazel }}
26-
build_flags:
27-
- '--cxxopt=/std:c++17'
28-
- '--host_cxxopt=/std:c++17'
29-
# Run using clang-cl
30-
- '--extra_execution_platforms=@protobuf//build_defs:x64_windows-clang-cl'
31-
- '--host_platform=@protobuf//build_defs:x64_windows-clang-cl'
32-
- '--extra_toolchains=@local_config_cc//:cc-toolchain-x64_windows-clang-cl'
33-
build_targets:
34-
- '@protobuf//:protobuf'
35-
- '@protobuf//:protobuf_lite'
36-
- '@protobuf//:protobuf_python'
37-
- '@protobuf//:protobuf_java'
38-
- '@protobuf//:protoc'
39-
- '@protobuf//:test_messages_proto2_cc_proto'
40-
- '@protobuf//:test_messages_proto3_cc_proto'
41-
421
bcr_test_module:
43-
module_path: "examples"
2+
module_path: examples
443
matrix:
45-
platform: ["debian10", "macos", "ubuntu2004"]
4+
platform: ["debian10", "macos", "ubuntu2004", "windows"]
465
bazel: [7.x]
6+
477
tasks:
48-
run_test_module:
49-
name: "Run test module"
8+
verify_targets:
9+
name: "Verify build targets"
5010
platform: ${{ platform }}
5111
bazel: ${{ bazel }}
52-
build_flags:
53-
- '--host_cxxopt=-std=c++17'
54-
- '--cxxopt=-std=c++17'
55-
build_targets:
56-
- "//..."
57-
# Separate windows for different c++ build flags.
58-
run_test_module_windows:
59-
name: "Run windows test module"
60-
platform: windows
61-
bazel: ${{ bazel }}
62-
build_flags:
63-
- '--cxxopt=/std:c++17'
64-
- '--host_cxxopt=/std:c++17'
6512
build_targets:
66-
- "//..."
13+
- '//...'
14+
- '@com_google_protobuf//:protobuf'
15+
- '@com_google_protobuf//:protobuf_lite'
16+
- '@com_google_protobuf//:protobuf_python'
17+
- '@com_google_protobuf//:protobuf_java'
18+
- '@com_google_protobuf//:protoc'
19+
- '@com_google_protobuf//:test_messages_proto2_cc_proto'
20+
- '@com_google_protobuf//:test_messages_proto3_cc_proto'

0 commit comments

Comments
 (0)