Skip to content
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

Revert "build: Skip empty protobuf files (#605)" #639

Merged
merged 3 commits into from May 4, 2022

Conversation

LucioFranco
Copy link
Member

This reverts commit c577d3f.

Fixes hyperium/tonic#964

@LucioFranco LucioFranco merged commit b0cbe9c into master May 4, 2022
@LucioFranco LucioFranco deleted the lucio/revert-skip-empty branch May 4, 2022 20:10
jgraettinger added a commit to estuary/flow that referenced this pull request May 23, 2022
Since its reversion of a commit which omits empty files:
tokio-rs/prost#639
jgraettinger added a commit to estuary/flow that referenced this pull request May 23, 2022
Since its reversion of a commit which omits empty files:
tokio-rs/prost#639
@tustvold
Copy link

tustvold commented Jul 5, 2022

I'm a little confused by this, reading hyperium/tonic#964 it seems like the issue is that if someone is specifying include_file("google.protobuf.rs") and if this file is stripped out because it is empty, the build will error. Is this actually a problem? The ticket says that removing the include_file eliminates the issue, so isn't the fix to just not add redundant include_file calls in build.rs?

I mainly ask because I'm adding workarounds for the empty files, and it would be nice to not need to apache/arrow-rs#2005

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.

After updating to v0.7 with prost-types v0.10 "google.protobuf.rs" is not available
2 participants