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
[deps] Upgrade protobuf version to v25.0 #34513
Conversation
@veblush What are the next steps to get the full suite of tests to run against this change? |
b59a540
to
a923292
Compare
This includes deleting the obsolete .upb.c files and renaming upb-generated to upb-gen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm thank you for taking care of this upgrade!
This needs another cut back. |
To mitigate the following windows long path issue found in #34513. Using `pyb` instead of `python_build` saves 9 characters. --- `Distribution Tests Python Windows` failed because of `T:\altsrc\github\grpc\workspace_python_windows_x86_Python38_32bit\python_build\\temp.win-amd64-cpython-312\\Release\\src\core\ext\upb-gen\envoy\extensions\load_balancing_policies\client_side_weighted_round_robin\v3\client_side_weighted_round_robin.upb_minitable.obj`
To mitigate the following windows long path issue found in grpc#34513. Using `pyb` instead of `python_build` saves 9 characters. --- `Distribution Tests Python Windows` failed because of `T:\altsrc\github\grpc\workspace_python_windows_x86_Python38_32bit\python_build\\temp.win-amd64-cpython-312\\Release\\src\core\ext\upb-gen\envoy\extensions\load_balancing_policies\client_side_weighted_round_robin\v3\client_side_weighted_round_robin.upb_minitable.obj`
This commit upgrades gRPC to protobuf v25.0 and makes some fixes to account for upb changes. One major change is that upb has been merged into the protobuf repo, so we can now drop the separate `@upb` dependency. Another is that `.upb.c` files no longer exist and there are new `.upb_minitable.h` and `.upb_minitable.c` files. The longer filenames exceeded a Windows restriction, so to work around that I renamed the `upb-generated` directory to just `upb-gen`, and likewise for `upbdefs-generated`.
This commit upgrades gRPC to protobuf v25.0 and makes some fixes to account for upb changes. One major change is that upb has been merged into the protobuf repo, so we can now drop the separate
@upb
dependency. Another is that.upb.c
files no longer exist and there are new.upb_minitable.h
and.upb_minitable.c
files. The longer filenames exceeded a Windows restriction, so to work around that I renamed theupb-generated
directory to justupb-gen
, and likewise forupbdefs-generated
.