-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
[main] Rebuild for protobuf423 while accounting for change in versioning scheme #188
Conversation
…nda-forge-pinning 2023.06.15.14.18.49
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do. This message was generated by GitHub actions workflow run https://github.com/conda-forge/bazel-feedstock/actions/runs/5363916604. |
For libprotobuf=4.23.2 builds now the error is:
This seems a classical error due to the fact that abseil is now a public dependency of libprotobuf and is not linked for this target. |
Linux is building. Remaining problems:
|
@conda-forge/bazel the PR is ready for review. |
Thanks for the cleanup! |
Let's get this forward! |
* parametrize versions correctly between protobuf for C++ & Java * Add abseil linkage for protobuf * Add C++17 to BAZEL_BUILD_OPTS Co-Authored-By: Silvio Traversaro <[email protected]>
* parametrize versions correctly between protobuf for C++ & Java * Add abseil linkage for protobuf * Add C++17 to BAZEL_BUILD_OPTS Co-Authored-By: Silvio Traversaro <[email protected]>
Starting with the v20.x protoc release, the versioning of C++ and Java libraries of protobuf changed, and while the minor.patch is ensured to be the same value, the major version are not related anymore, see https://protobuf.dev/support/version-support/ . So we need to manually handle the protobuf-java major version.
Fortunatly, for both libprotobuf 3.21 ad 4.23.2 the corresponding protobuf-java major version is the same (3), so we do not need specific logic to select the protobuf-java major version.
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)