-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
Migrate utf8_range dependency to a git subtree #11608
Migrate utf8_range dependency to a git subtree #11608
Conversation
git-subtree-dir: third_party/utf8_range git-subtree-split: 72c943dea2b9240cd09efde15191e144bc7c7d38
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.
I think this should land internally first or maybe copybara needs to be fixed after-the-fact since utf8_range.cmake isn't github only. Also just sanity checking that this doesn't affect how we use the utf8_range submodule for php release (https://github.com/protocolbuffers/protobuf/blob/main/php/BUILD.bazel#L34)? (pretty sure we have test coverage for this though)
This shouldn't affect bazel builds at all, since we depend on utf8_range through Bazel. I plan on importing this with copybara, it should work? I might have to force it in tho :P |
This will make the CMake workflow easier, where users won't need to either install utf8_range or initialize our submodules. Since this is a relatively small library that's owned by us, we can just bundle it with protobuf as a subtree Closes #11608 COPYBARA_INTEGRATE_REVIEW=#11608 from mkruskal-google:utf8_range_dep 57d5d91 FUTURE_COPYBARA_INTEGRATE_REVIEW=#11608 from mkruskal-google:utf8_range_dep 57d5d91 PiperOrigin-RevId: 503304867
This will make the CMake workflow easier, where users won't need to either install utf8_range or initialize our submodules. Since this is a relatively small library that's owned by us, we can just bundle it with protobuf as a subtree