Enable CI/CQ-based symbol prefixing verification on all Win shared builds. Bug: 543471701 Change-Id: I9c5b01ca45161b241d119ceed01451ed6a6a6964 Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/100551 Auto-Submit: Rudolf Polzer <rpolzer@google.com> Reviewed-by: Xiangfei Ding <xfding@google.com> Presubmit-BoringSSL-Verified: boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Commit-Queue: Xiangfei Ding <xfding@google.com>
diff --git a/infra/config/generated/cr-buildbucket.cfg b/infra/config/generated/cr-buildbucket.cfg index 3ee841c..5cf1b62 100644 --- a/infra/config/generated/cr-buildbucket.cfg +++ b/infra/config/generated/cr-buildbucket.cfg
@@ -1915,6 +1915,7 @@ cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" properties_j: "$gatekeeper:{\"group\":\"client.boringssl\"}" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x86\"" properties_j: "prefixed_symbols:true" @@ -2235,6 +2236,7 @@ cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" properties_j: "$gatekeeper:{\"group\":\"client.boringssl\"}" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x64\"" properties_j: "prefixed_symbols:true" @@ -3817,6 +3819,7 @@ name: "boringssl" cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x86\"" properties_j: "prefixed_symbols:true" @@ -3835,6 +3838,7 @@ name: "boringssl" cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x86\"" properties_j: "prefixed_symbols:true" @@ -4073,6 +4077,7 @@ name: "boringssl" cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x64\"" properties_j: "prefixed_symbols:true" @@ -4091,6 +4096,7 @@ name: "boringssl" cipd_package: "infra/recipe_bundles/chromium.googlesource.com/chromium/tools/build" cipd_version: "refs/heads/main" + properties_j: "check_prefixed_symbols:true" properties_j: "cmake_args:{\"BUILD_SHARED_LIBS\":\"1\"}" properties_j: "msvc_target:\"x64\"" properties_j: "prefixed_symbols:true"
diff --git a/infra/config/main.star b/infra/config/main.star index b75bcb5..f35fce2 100755 --- a/infra/config/main.star +++ b/infra/config/main.star
@@ -1481,6 +1481,7 @@ "BUILD_SHARED_LIBS": "1", }, "prefixed_symbols": True, + "check_prefixed_symbols": True, "upload_to_cas": FINISHED_OUTPUT_FILES_WIN_SHARED, }, ) @@ -1626,6 +1627,7 @@ "BUILD_SHARED_LIBS": "1", }, "prefixed_symbols": True, + "check_prefixed_symbols": True, "upload_to_cas": FINISHED_OUTPUT_FILES_WIN_SHARED, }, )