-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add ibc-go-sdk-50-simapp
#151
Conversation
This reverts commit b1deb8c.
@JonathanLorimer Any idea what might be causing this error? Haven't been able to reproduce it outside of Nix, whether on macOS or Linux...
|
@romac |
This is probably my lack of understanding of the go module system, but maybe nix things that require is pointing to the link I posted, but its actually meant to resolve to this https://github.com/cosmos/ibc-go/tree/modules/capability/v1.0.0-rc3/modules |
forgive my ignorance, but if you are actually supposed to split tags like that in the go.mod file and its supposed to map to |
The error message is kinda true the module name doesn't match up https://github.com/cosmos/ibc-go/blob/upgrade-sdk-v0.50-alpha1/modules/capability/go.mod#L1 TBH I am having a really hard time following what version of go everything in this repo is on / what uses go.mod and what doesn't. And when I
|
Thanks for the investigation on this issue, @JonathanLorimer. It's very weird: ibc-go imports the capability module and the capability module has its own go.mod so there shouldn't be any |
Superseded by #160 for IBC Go v8-alpha.1. Unfortunately the build still fails with a similar error: #160 (comment) |
Subdirectories that contain their own I think this is more of an issue of code organization in Go projects like ibc-go. Ideally, each Go module should be placed in their own subdirectory within the project, instead of nesting one module within the subdirectory of another. Doing so would also make it easier for external users to discover what Go modules are offered by a Go project by simply scanning a root directory. |
Not for merge, only testing for now.
Had to update the flake inputs to get Go 1.20 final instead of pre-release, but had to remove old packages whose source can't be found anymore for it to work.