commit fb1e7693e2589d4c089b7bbaae082b4362b3a8b3
author David Benjamin <davidben@google.com> Tue Jan 21 11:16:55 2025 -0500
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jan 22 09:10:31 2025 -0800
tree 2f2acbf6712f4478740f504b78e7c8d69d62ee8d
parent 1c40397b45015f836ff7943f20012e93facfda36

Sync presubmit.yml with what landed in BCR

Bazel 7 defaults to C++14, and BCR doesn't pick up .bazelrc, so we need
to specify this in yet another place.

See also
https://github.com/bazelbuild/bazel-central-registry/pull/3579

Change-Id: I5e639b86ec52dd9ebd14c58aa9cca7997a5f5c87
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/75407
Commit-Queue: Bob Beck <bbe@google.com>
Reviewed-by: Bob Beck <bbe@google.com>
Auto-Submit: David Benjamin <davidben@google.com>

.bcr/presubmit.yml

diff --git a/.bcr/presubmit.yml b/.bcr/presubmit.yml
index 32813ba..44e4b7a 100644
--- a/.bcr/presubmit.yml
+++ b/.bcr/presubmit.yml
@@ -3,17 +3,25 @@
   - "@boringssl//:ssl"
 
 matrix: &matrix
-  non_macos_platform:
+  linux_platforms:
     - debian11
     - ubuntu2204
-    - windows
   bazel: [7.x]
 tasks:
-  verify_targets:
-    name: Verify build targets
-    platform: ${{ non_macos_platform }}
+  verify_targets_on_linux:
+    name: Verify build targets on Linux
+    platform: ${{ linux_platforms }}
     bazel: ${{ bazel }}
     build_targets: *build_targets
+    build_flags:
+      - '--cxxopt=-std=c++17'
+  verify_targets_on_windows:
+    name: Verify build targets on Windows
+    platform: windows
+    bazel: ${{ bazel }}
+    build_targets: *build_targets
+    build_flags:
+      - '--cxxopt=/std:c++17'
   # Work around https://github.com/bazelbuild/bazel/issues/10472
   verify_targets_on_macos:
     name: Verify build targets on macOS
@@ -27,12 +35,22 @@
   module_path: util/bazel-example
   matrix: *matrix
   tasks:
-    run_test_module:
-      name: Run test module
-      platform: ${{ non_macos_platform }}
+    run_test_module_on_linux:
+      name: Run test module on Linux
+      platform: ${{ linux_platforms }}
       bazel: ${{ bazel }}
       build_targets:
         - //...
+      build_flags:
+        - '--cxxopt=-std=c++17'
+    run_test_module_on_windows:
+      name: Run test module on Windows
+      platform: windows
+      bazel: ${{ bazel }}
+      build_targets:
+        - //...
+      build_flags:
+        - '--cxxopt=/std:c++17'
     # Work around https://github.com/bazelbuild/bazel/issues/10472
     run_test_module_on_macos:
       name: Run test module on macOS