-
-
Notifications
You must be signed in to change notification settings - Fork 631
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
Can't get this repo #776
Comments
Could you try with |
Scratch that please try |
both of them still not working. |
What version of Go are you using? |
Try to break circular module dependency loop: anacrolix/torrent#776
Wow, this really sucks. Even updating dependencies on anacrolix/torrent to a version that doesn't depend on dht versions that don't depend on older anacrolix/torrent doesn't work. The Go tooling here is appalling. I'm at a loss. |
Note that with the default Fortunately, as of Go 1.17 we do have two features that should help. One is module graph pruning. To activate it, run The other is better semantics for
Ideally, you can combine |
Thank you so much @bcmills! |
@bcmills this doesn't actually work from the library side: If a potential downstream runs |
go version go1.16.15 darwin/arm64 |
There is another error:
|
@Shawn-Huang-Tron net/netip was only added in go 1.18, so you must use that. |
Is that means that I have to upgrade my go version to 1.18? |
@Shawn-Huang-Tron, per the Go release policy, the only major releases of the toolchain currently supported by the Go project are 1.18 (currently 1.16.6) and 1.19 (currently 1.19.1). You should upgrade either way so that you pick up current security fixes. 😅 |
Hmm, it looks like you've already got graph pruning enabled — and, even more confusingly, once the
|
This looks to me like a bug in |
Thank you @Shawn-Huang-Tron for putting the effort in to report this. |
This seems to be unreproducible with go1.20.3. I don't think a patch in golang/go has arrived, but suspect some dependencies have been shuffled around in newer versions of anacrolix/torrent and anacrolix/dht that avoid the problem. Please reopen if the issue comes up again. |
The text was updated successfully, but these errors were encountered: