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

Use Actions to update Go version automatically. #51

Open
wants to merge 56 commits into
base: master
Choose a base branch
from

Conversation

capric98
Copy link

@capric98 capric98 commented Mar 5, 2021

No description provided.

@capric98
Copy link
Author

capric98 commented Mar 5, 2021

I was about to open a new merge request to change the download url of Go, but I committed it to master branch heedlessly so here it is...
Compare to storage.googleapis.com, dl.google.com seems to have a better CDN in some areas.

@canha
Copy link
Owner

canha commented Mar 12, 2021

Hi @capric98, thanks for submitting this!

Wouldn't it be more beneficial to introduce this logic into the script itself? We would need to find a way to do it without Python, but what do you think?

@capric98
Copy link
Author

It is a little bit hard since bash does not support PCRE natively. To achieve this, we have to introduce additional tools(such as Perl/Python), or, write a bash function in Unix way, which is difficult...
I tried to get latest version from git, which is complex(they released 1.16.2/1.15.10 at same time...)
Additionally, golang.org got blocked by GFW and cannot be access from Mainland China, while most web approach redirect to https://golang.org/dl.

I am also waiting for a better solution :)

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

Successfully merging this pull request may close these issues.

None yet

2 participants