Skip to content

Bump prost-build to 0.10 to solve protobuf version incompatibility #2

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

Merged
merged 1 commit into from
May 30, 2022

Conversation

jvanstraten
Copy link
Collaborator

@jvanstraten jvanstraten commented May 26, 2022

Should help solve the issue described here and here by effectively upgrading protoc from 3.15.8 to 3.19.4. However, from now on either protoc needs to already be installed on the system to install, or prost-build needs to have the tools available to build it from source (needs CMake an C++ toolchain).

ETA: Rerunning CI on my fork shows that this has actually broken again for no apparent reason, turning into draft. forgot to increment one of instances of the dependency, no idea how it ever passed before this 🤷

Should help solve the issue described in [1] by effectively upgrading
protoc from 3.15.8 to 3.19.4. However, from now on either protoc needs
to already be installed on the system to install, or prost-build needs
to have the tools available to build it from source (needs CMake an
C++ toolchain).

[1] protocolbuffers/protobuf#10051
@jvanstraten jvanstraten marked this pull request as ready for review May 30, 2022 17:24
@jacques-n jacques-n merged commit 644425f into substrait-io:main May 30, 2022
@jvanstraten jvanstraten deleted the bump-prost branch May 30, 2022 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants