镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Update rbenv / ruby-build - #1913

Merged
asottile merged 1 commit into
pre-commit:masterfrom
jalessio:jamie/upgrade-ruby
May 18, 2021
Merged

asottile merged 1 commit into
pre-commit:masterfrom
jalessio:jamie/upgrade-ruby

Conversation

@jalessio

@jalessio jalessio commented May 15, 2021 •

Copy link
Copy Markdown
Contributor

Similar to PR #1614, updates rbenv and ruby-build to their latest releases which makes more recent versions of Ruby available to pre-commit.

rbenv

Doesn't look like rbenv has any recent tags (latest is from March 25, 2019), so this is current HEAD of master branch:
rbenv/rbenv@585ed84

ruby-build 20210510

https://github.com/rbenv/ruby-build/releases/tag/v20210510

@jalessio

jalessio commented May 15, 2021 •

Copy link
Copy Markdown
Contributor Author

@asottile per your mention about "reproducible builds" for the tar.gz files it could be accomplished in a Github-specific way by updating make-archives to grab the URLs like this instead of doing a git clone:

https://github.com/rbenv/rbenv/archive/585ed84283f3308380b843391ee7b12706ecff87.tar.gz

Looks like it also supports short shas (but the resulting file is the full git sha):
https://github.com/rbenv/ruby-build/archive/e9fa4bf.tar.gz

It also looks like calling that repeatedly produces a tar file with the same sha1sum so in a way that is "reproducible" and can be added to CI checks.

This would work for the given set of REPOS since they're all hosted on Github, but wouldn't work if other git hosting providers are added to that list.

@asottile

Copy link
Copy Markdown
Member

I'll rebase this on #1915 once that merges!

@asottile
asottile force-pushed the jamie/upgrade-ruby branch from 331cc9d to 14afbc7 Compare May 18, 2021 15:08
@asottile
asottile merged commit a1f2d69 into pre-commit:master May 18, 2021
@jalessio
jalessio deleted the jamie/upgrade-ruby branch May 18, 2021 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants