-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
runtime: netpoll_windows fails to build on windows-arm #27686
Comments
Someone needs to generate internal/poll/defs_windows_arm.go. Isn't this just a subset of #26148? And in fact CL 128715 should fix this. Do we need a separate issue for this? |
No idea. I was just looking at the builders and assuming that anything present with “show only first-class ports” checked should pass. If we're adding new builders for ports that don't work yet, arguably they should not be classified as “first-class”. Perhaps the windows-arm builder is misclassified? |
As far as I can tell, the fix is in https://go-review.googlesource.com/c/go/+/128715 It is up to us to +2 and submit it. Ian asked for some changes, which Jordan replied to. Ian, please, +2 and submit, if you are OK with CL. Thank you. Alex |
CL 128715 was submitted. Current build https://build.golang.org/log/391237e5d9a43cb80e9d427b8046dc9e29157f9d gets past the
I do not think there is anything to do here. Making remaining parts of windows-arm port will be tracked in #26148 Alex |
From https://build.golang.org/log/9ce72620b62df52a4a11332df8cf40c65f23a798:
CC: @alexbrainman @ianlancetaylor @aclements @randall77
The text was updated successfully, but these errors were encountered: