-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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
cmd/go: 1.18 fails when vendoring and main module replaces itself #51285
Comments
go list
on containerd repo
related #34417 simpler reproducer:
results:
|
Duplicate of #34417 |
Looking at the specific example of
As of Go 1.17, the appropriate way to prune out cyclic dependencies is to update the For compatibility with older |
These versions are compatible with Go 1.18. There are a lot of OSS bugs about this issue. Notable discussions: - golang/go#51285 - containerd/containerd#6586 This was fixed upstream in the main branch in containerd/containerd#6605. The fix was back-ported to v1.5 and v1.6 in containerd/containerd#6716 and containerd/containerd#6717 respectively. These backports are available in the following versions: ```shell /tmp/gopathJJGZf/containerd$ git tag --contains 765df66099eec88d0365eaa1e9a933877a058f0b v1.6.2 v1.6.3 v1.6.4 v1.6.5 v1.6.6 v1.6.7 v1.6.8 /tmp/gopathJJGZf/containerd$ git tag --contains 86bec213720b54bf9ea2813e2c2a577271af58d3 v1.5.11 v1.5.12 v1.5.13 ``` So use the smallest versions out of them. PiperOrigin-RevId: 480965567
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
go list
in root dirWhat did you expect to see?
Like go1.17, it succeeds with:
What did you see instead?
The text was updated successfully, but these errors were encountered: