-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
go 1.17.7 #94895
go 1.17.7 #94895
Conversation
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. To keep this pull request open, add a |
Fix for |
Fix for |
Fix for |
I have checked out the branch and installed 1.17.7 in my local, |
Any chance for the new build run to happen? |
823bb8c
to
bb1a5ec
Compare
7 formula failed to build, and all look to me to non-go related errors:
So I think upgrading the go version is a good call. |
Thanks for the list. That's only for the ARM part, more might pop up on Intel or Linux. Unless this release fixes a CVE and there is an emergency, we should focus on fixing all of these. We can't just stay forever with broken builds. We have a policy in place, and we should try to follow it. Here a patches for: |
@iMichka not sure if these count as emergencies, but here are the CVEs fixed in Go 1.17.7:
|
|
This should be tried again now that everything was fixed and virgil marked as depending on x86_64 in #96009 |
Yes. We also fixed goproxy: #96014 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's ship this when CI finishes.
At least I got some tests green ... |
Created with
brew bump-formula-pr
.resource
blocks may require updates.