Skip to content

fix: update MODULE.bazel.lock for protobuf v34.0.bcr.1#6456

Merged
johanbrandhorst merged 2 commits intorenovate/major-protobuf-monorepofrom
copilot/sub-pr-6422-again
Mar 10, 2026
Merged

fix: update MODULE.bazel.lock for protobuf v34.0.bcr.1#6456
johanbrandhorst merged 2 commits intorenovate/major-protobuf-monorepofrom
copilot/sub-pr-6422-again

Conversation

Copy link
Contributor

Copilot AI commented Mar 10, 2026

The CI bazel job fails because MODULE.bazel.lock still references protobuf v33.5 after the dependency was bumped to v34.0.bcr.1 in MODULE.bazel.

Changes

  • MODULE.bazel.lock: Replace protobuf/33.5 entries with protobuf/34.0.bcr.1 (SHA256 hashes verified against BCR); remove stale source.json entries for nlohmann_json/3.6.1, rules_apple/4.1.0, rules_swift/3.1.2, and swift_argument_parser/1.3.1.2 that are no longer referenced

Notes

PR #6428 attempted the same fix but also added a single_version_override patch to address CcToolchainConfigInfo not being a global in Bazel 9.0.0 — this caused a "redeclared at top level" error because the BCR package 34.0.bcr.1 already ships with a load.patch applying exactly that fix. No additional patching is needed here.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Replace protobuf/33.5 entries with protobuf/34.0.bcr.1 entries and
remove stale source.json entries that are no longer referenced.

The BCR version 34.0.bcr.1 already contains an upstream patch (load.patch)
that fixes the CcToolchainConfigInfo compatibility issue with Bazel 9.0.0,
so no additional patching is needed.

Co-authored-by: johanbrandhorst <6604151+johanbrandhorst@users.noreply.github.com>
Copilot AI changed the title [WIP] Update dependency protobuf to v34 fix: update MODULE.bazel.lock for protobuf v34.0.bcr.1 Mar 10, 2026
@johanbrandhorst johanbrandhorst marked this pull request as ready for review March 10, 2026 19:17
@johanbrandhorst johanbrandhorst merged commit b6dfbe8 into renovate/major-protobuf-monorepo Mar 10, 2026
14 checks passed
@johanbrandhorst johanbrandhorst deleted the copilot/sub-pr-6422-again branch March 10, 2026 19:18
johanbrandhorst added a commit that referenced this pull request Mar 10, 2026
* chore(deps): update dependency protobuf to v34

* fix: update MODULE.bazel.lock for protobuf v34.0.bcr.1 (#6456)

* Initial plan

* fix: update MODULE.bazel.lock for protobuf v34.0.bcr.1

Replace protobuf/33.5 entries with protobuf/34.0.bcr.1 entries and
remove stale source.json entries that are no longer referenced.

The BCR version 34.0.bcr.1 already contains an upstream patch (load.patch)
that fixes the CcToolchainConfigInfo compatibility issue with Bazel 9.0.0,
so no additional patching is needed.

Co-authored-by: johanbrandhorst <6604151+johanbrandhorst@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: johanbrandhorst <6604151+johanbrandhorst@users.noreply.github.com>

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: johanbrandhorst <6604151+johanbrandhorst@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants