Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: automate publishing releases to Bazel Central Registry #16014

Closed
wants to merge 4 commits into from
Closed

Conversation

alexeagle
Copy link
Collaborator

Note, recent releases on the BCR have a patch set applied, and it seems these patches are developed independently to "fix" each protobuf release, rather than make changes to protobuf repo.

The effect of this PR will be to create a broken publish to BCR for each protobuf release. At least this red PR on BCR will be our indication that the patches need to be manually replayed there.

In parallel, starting with #16013 I'll apply as many of those patches to the protobuf repo as possible. That will reduce the manual effort for each release.

Replaces #14565 which originated from my fork so the tests wouldn't run.

Note, recent releases on the BCR have a patch set applied, and it seems these patches are developed
independently to "fix" each protobuf release, rather than make changes to protobuf repo.

The effect of this PR will be to create a *broken* publish to BCR for each protobuf release.
At least this red PR on BCR will be our indication that the patches need to be manually replayed there.

In parallel, starting with #16013 I'll apply as many of those patches to the protobuf repo as possible.
That will reduce the manual effort for each release.
alexeagle added a commit to bazelbuild/rules_proto that referenced this pull request Mar 4, 2024
This allows rules_proto to supply a pre-built binary toolchain for protoc without needing any Bazel module supplied by the protocolbuffers maintainers.
Obviates the need for protobuf to be released to the BCR as in protocolbuffers/protobuf#16014
Allows the protocolbuffers team to stop caring about Bazel builds for external users, and to close associated bugs.

Next step in rules_proto supplying a toolchain that doesn't require users build protoc from sources.
alexeagle added a commit to bazelbuild/rules_proto that referenced this pull request Mar 4, 2024
This allows rules_proto to supply a pre-built binary toolchain for protoc without needing any Bazel module supplied by the protocolbuffers maintainers.
Obviates the need for protobuf to be released to the BCR as in protocolbuffers/protobuf#16014
Allows the protocolbuffers team to stop caring about Bazel builds for external users, and to close associated bugs.

Next step in rules_proto supplying a toolchain that doesn't require users build protoc from sources.
alexeagle added a commit to bazelbuild/rules_proto that referenced this pull request Mar 4, 2024
This allows rules_proto to supply a pre-built binary toolchain for protoc without needing any Bazel module supplied by the protocolbuffers maintainers.
Obviates the need for protobuf to be released to the BCR as in protocolbuffers/protobuf#16014
Allows the protocolbuffers team to stop caring about Bazel builds for external users, and to close associated bugs.

Next step in rules_proto supplying a toolchain that doesn't require users build protoc from sources.
@anandolee anandolee added the bazel label Mar 7, 2024
.bcr/config.yml Outdated Show resolved Hide resolved
.bcr/presubmit.yml Outdated Show resolved Hide resolved
.bcr/presubmit.yml Outdated Show resolved Hide resolved
.bcr/presubmit.yml Outdated Show resolved Hide resolved
deannagarcia pushed a commit to deannagarcia/protobuf that referenced this pull request Jun 20, 2024
…lbuffers#16014)

Note, recent releases on the BCR have a patch set applied, and it seems these patches are developed independently to "fix" each protobuf release, rather than make changes to protobuf repo.

The effect of this PR will be to create a *broken* publish to BCR for each protobuf release. At least this red PR on BCR will be our indication that the patches need to be manually replayed there.

In parallel, starting with protocolbuffers#16013 I'll apply as many of those patches to the protobuf repo as possible. That will reduce the manual effort for each release.

Replaces protocolbuffers#14565 which originated from my fork so the tests wouldn't run.

Closes protocolbuffers#16014

COPYBARA_INTEGRATE_REVIEW=protocolbuffers#16014 from protocolbuffers:bcr e17d9c8
PiperOrigin-RevId: 615026796
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants