Skip to content
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

Make ensure-go-installed work again (on macOS) #1129

Closed
wants to merge 2 commits into from

Conversation

staktrace
Copy link

@staktrace staktrace commented May 30, 2022

PR #966 updated the PREFERRED_GO_VERSION from go1.14.7
to go1.16.4 but didn't update the corresponding binary
checksums. The checksums can be found at https://go.dev/dl/

Additionally, the package file inside has changed and
requires updating.

Fixes issue: #1128

PR github#966 updated the PREFERRED_GO_VERSION from go1.14.7
to go1.16.4 but didn't update the corresponding binary
checksums. The checksums can be found at https://go.dev/dl/

Additionally, the package file inside has changed and
requires updating.
@staktrace staktrace changed the title Update preferred version checksums Make ensure-go-installed work again (on macOS) May 30, 2022
@timvaillancourt timvaillancourt added this to the v1.1.5 milestone Jun 2, 2022
@timvaillancourt timvaillancourt self-requested a review June 2, 2022 21:41
Copy link
Collaborator

@timvaillancourt timvaillancourt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@staktrace d'oh 🤦. Thanks for this PR!

New checksums LGTM 👍:

tim@Tims-MacBook-Pro gh-ost % curl -sLo - https://dl.google.com/go/go1.16.4.darwin-amd64.pkg | shasum -a 256  
9f9b940d0f4b3ac764f0a33d78384a87b804aab29d1aacbdc9bca3a3480e9272  -
tim@Tims-MacBook-Pro gh-ost % curl -sLo - https://dl.google.com/go/go1.16.4.linux-amd64.tar.gz | shasum -a 256
7154e88f5a8047aad4b80ebace58a059e36e7e2e4eb3b383127a28c711b4ff59  -

@timvaillancourt timvaillancourt linked an issue Jun 2, 2022 that may be closed by this pull request
@timvaillancourt timvaillancourt requested review from dm-2 and rashiq June 2, 2022 21:44
@timvaillancourt
Copy link
Collaborator

cc @dm-2 / @rashiq as I can't merge this due to the outstanding gh-ost-build-deploy-tarball CI

@timvaillancourt
Copy link
Collaborator

Another PR in the next release duplicates this. Closing in favour of #1136 👍

Thanks for the issue and PR 🙇

@dm-2 dm-2 removed this from the v1.1.5 milestone Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

go binary checksums are out of date
3 participants