Mess around with proto files in a probably doomed attempt to resolve relayer issues #2020
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.
When running the relayer, we get an error that
This is an entirely self-inflicted issue; there's no actual problem here, other than the totally unreasonable requirement that every
.proto
file across all source code in the dependency tree has a unique filename.To paraphrase Werner Herzog,
As we have chosen to use the Go relayer, we have our share of this curse. Let us examine it.
suggesting that the IBC package depends on ibc-go-vendor, even though the
penumbra
module'sbuf.yaml
specifiesThis is weird. In an attempt to figure out what's going on, try removing the
ibc-go-vendor
package entirely, to see what's happening.In the future, we should move all of the Rust imports to use the proto files pulled from Buf, but this requires either (a) telling
prost
/tonic
about where to find the proto files in the Buf module cache or (b) telling Buf how to drive prost and tonic. (a) seems easier but for now, we can just accept alittle bit of divergence.