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

dl: add go1.2.2, go1.3.3 and go1.4.3 #45608

Open
perillo opened this issue Apr 17, 2021 · 5 comments
Open

dl: add go1.2.2, go1.3.3 and go1.4.3 #45608

perillo opened this issue Apr 17, 2021 · 5 comments

Comments

@perillo
Copy link
Contributor

@perillo perillo commented Apr 17, 2021

These versions don't have the SH256 checksum available, however the SHA1 checkum is listed on https://golang.org/dl, as an example https://golang.org/dl/#go1.4.3.

In order to support them, it is necessary to print a warning in case the SHA256 is not available, instead of terminating the program.
Here is an example of the warning from my CL (not yet submitted):

Warning: SHA256 Checksum not found: https://storage.googleapis.com/golang/go1.4.3.linux-amd64.tar.gz.sha256: 404 Not Found
The SHA1 Checksum can be found on https://golang.org/dl/#go1.4.3
@toothrot toothrot changed the title dl: add go1.2.2, go1.3.3 and go1.4.3 x/dl: add go1.2.2, go1.3.3 and go1.4.3 Apr 19, 2021
@gopherbot gopherbot added this to the Unreleased milestone Apr 19, 2021
@toothrot
Copy link
Contributor

@toothrot toothrot commented Apr 19, 2021

Loading

@FiloSottile
Copy link
Contributor

@FiloSottile FiloSottile commented Apr 20, 2021

We can and should just replace SHA1 hashes with SHA256 hashes for older releases, instead.

Loading

@perillo
Copy link
Contributor Author

@perillo perillo commented May 6, 2021

@FiloSottile, any news about adding SHA256 hashes for older releases?

Loading

@dmitshur
Copy link
Contributor

@dmitshur dmitshur commented May 21, 2021

@perillo It's unlikely we'll have time for this while in the Go 1.17 freeze, but perhaps afterwards.

Loading

@dmitshur dmitshur changed the title x/dl: add go1.2.2, go1.3.3 and go1.4.3 dl: add go1.2.2, go1.3.3 and go1.4.3 May 21, 2021
@perillo
Copy link
Contributor Author

@perillo perillo commented May 22, 2021

@dmitshur, well, it is not really an important change. Thanks for the reply.

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants