-
Notifications
You must be signed in to change notification settings - Fork 17.6k
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: go run golang.org/x/example@<version>
panics when run in a workspace
#51390
Comments
(I wonder if this is a bad interaction between
(That is: |
Indeed, reproduced by adding a
|
The fix, I think, is to make |
Oof, apologies! I would have reported that I had a |
Change https://go.dev/cl/388917 mentions this issue: |
Change https://go.dev/cl/389575 mentions this issue: |
This is merged at tip, and can be fixed by merging the 1.18 backport whenever it has a +2 and passing TryBot run. |
…rsion For #51390 Change-Id: I805e66809b2aafb48f7040dee72910dd7d6c1396 Reviewed-on: https://go-review.googlesource.com/c/go/+/388917 Trust: Bryan Mills <bcmills@google.com> Run-TryBot: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Michael Matloob <matloob@golang.org>
go run golang.org/x/example@<version>
panics when run in a workspace
…ackage at a specified version Fixes #51390 Change-Id: I805e66809b2aafb48f7040dee72910dd7d6c1396 Reviewed-on: https://go-review.googlesource.com/c/go/+/388917 Trust: Bryan Mills <bcmills@google.com> Run-TryBot: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Michael Matloob <matloob@golang.org> (cherry picked from commit 86b5f6a) Reviewed-on: https://go-review.googlesource.com/c/go/+/389575 Trust: Dmitri Shuralyov <dmitshur@golang.org> Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Closed by merging 9f40b4f to release-branch.go1.18. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
No.
What did you do?
cd typeparams/example
go run golang.org/x/example/gotypes@master generic-go-types.md
What did you expect to see?
Successful execution.
What did you see instead?
This command succeeds using
go1.17.1
, but panics at tip. Maybe I'm unaware of some change in semantics, but since I get a panic I suspect this is a bug.CC @bcmills @matloob
The text was updated successfully, but these errors were encountered: