Enable to build protobuf v30 with Bazel+MSVC #1178
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As seen in the following thread, the protobuf team is planning to stop supporting the combination of Bazel and cl.exe due to well-known its path length limitation. * protocolbuffers/protobuf#20085 As a preparation before switching to protobuf v30 (google#1177), let's explicitly add a commandline option as explained to continue using this combination as a short term solution. This commit should have no impact on Probobuf v29.x.
|
We have merged your PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
As seen in the following thread, the protobuf team is planning to stop supporting the combination of Bazel and cl.exe due to well-known its path length limitation.
As a preparation before switching to protobuf v30 (#1177), let's explicitly add a command line option as explained to continue using this combination as a short term solution.
This commit should have no impact on probobuf v29.x.
Issue IDs
third_party/protobufto v30.0 #1177Steps to test new behaviors (if any)
A clear and concise description about how to verify new behaviors (if any).
MODULE.bazelto use protobuf v30.rc1Additional context
Add any other context about the pull request here.