push
github
Upgrade to Protobuf 35.1 (#12876) Per https://github.com/protocolbuffers/protobuf/releases ``` okshiva@okshiva-mac2 grpc-java % curl -sL https://github.com/protocolbuffers/protobuf/releases/download/v35.1/protobuf-35.1.tar.gz | shasum -a 256 f0b6838e7368f3024ac8fd03abd0a1ac4 ``` While upgrading Protobuf to 35.1, two secondary Bazel dependencies (`rules_proto` and `rules_android`) had to be addressed to keep the CI green. Here is the context: 1. Added `rules_proto` to `repositories.bzl` and `MODULE.bazel`. The googleapis dependency requires `rules_proto` to build. During the CI job (--enable_bzlmod=false), the build failed during the analysis phase because googleapis could no longer implicitly fetch `rules_proto` via Protobuf. 2. Upgraded to rules_android v0.7.3 Previous version of `rules_android` (v0.7.2) explicitly hardcoded "`com.google.protobuf:protobuf-java:4.33.4`" in its internal maven artifacts. During the Android CI job, Bazel used our newly upgraded 35.1 compiler to generate internal Android resource files (gencode 4.35.1). However, because `rules_android`'s internal tools (like `ResourceProcessorBusyBox`) were locked to the 4.33.4 runtime, it triggered a `ProtobufRuntimeVersionException` (poison pill).
36525 of 41099 relevant lines covered (88.87%)
0.89 hits per line