-
Notifications
You must be signed in to change notification settings - Fork 18k
cmd/go: return an error for unsupported GOOS/GOARCH pair #12272
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
Comments
How about we make cmd/go print "perhaps unsupported platform?" message if
some core std packages (runtime, syscall and net?) have build errors?
Then we don't nees to maintain a list of supported platforms.
or we can detect whether there exists runtime/defs_$GOOS_$GOARCH.go file to
see if the platform is supported.
(I'm a little sad about the recent duplication of the list of cgo-enabled
platforms in cmd/dist. I even want to suggest that we make cmd/dist
generate the list for go/build)
|
It won't help the users if the error message is uncertain, they will end up in the issue tracker or the mailing list to ask about the case. Maintaining a single source of truth for the supported combinations are rather easy and explicit. |
I have another proposal.
We store the list of supported platforms in cmd/dist, and have it generate
a list of supported platforms for cmd/go and a list of cgo enabled
platforms for go/build.
|
Not critical for Go 1.6. |
CL https://golang.org/cl/22838 mentions this issue. |
Funny enough, we relied on the old behaviour. We use |
Okay. I'll reopen the issue to take another look. @bradfitz Is this WAI or do we need to change or revert the check? |
Yes, I think we do need to fix this. We should not issue the error for |
On it. |
CL https://golang.org/cl/24864 mentions this issue. |
Incredible turnaround time on this one, thanks! |
This bug is about cmd/go returning a useful error message when an invalid GOOS/GOARCH pair is used.
See confusion in #11908.
Related request in #12270 (if we fix this bug, we should probably fix both, using the same source of truth)
The text was updated successfully, but these errors were encountered: