-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
Portable Ruby 3.3.1 #17312
Portable Ruby 3.3.1 #17312
Conversation
If we're deleting 3.1 vendored gems may need to bump HOMEBREW_REQUIRED_RUBY_VERSION, which is technically a breaking change but let's pretend we didn't miss 4.3.0. |
52857ab
to
d51ae11
Compare
Yeh, might just retrospectively add to the notes there if I can get this merged 🔜 and note it'll land in 4.3.1. |
137e5e4
to
1854c04
Compare
Use the latest version of Portable Ruby.
Library/Homebrew/cask/utils.rb
Outdated
p.rmtree | ||
FileUtils.rm_r p |
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.
Pathname#rmtree
now never returning errors is a horrifically subtle change that questions the safety of every #rmtree
call we have, given the new behaviour now means rmtree
can return without actually removing the directory (somewhat of a security risk potentially).
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.
e.g. this error catching now doesn't work:
brew/Library/Homebrew/reinstall.rb
Lines 73 to 78 in 97030b7
backup_path(keg).rmtree if backup_path(keg).exist? | |
rescue Errno::EACCES, Errno::ENOTEMPTY | |
odie <<~EOS | |
Could not remove #{backup_path(keg).parent.basename} backup keg! Do so manually: | |
sudo rm -rf #{backup_path(keg)} | |
EOS |
We could override the behaviour in extend/pathname.rb
. Seems hacky but might be the safest solution.
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.
Yeh, good catch and idea here @Bo98.
7a6246a
to
ddb0c39
Compare
Seeing the same test failure in 10.13 and up that was showing in CI earlier:
10.11 and 10.12 weren't able to install the required gems for |
I assume this was the case with Ruby 3.1.4 too? |
6e5af63
to
9576a7b
Compare
9576a7b
to
727cf5e
Compare
Thanks for all the help here @Bo98 🎉 |
Use the latest version of Portable Ruby.
I may have missed some stuff here.
May need new checksums once https://github.com/Homebrew/homebrew-portable-ruby/actions/runs/9105311684 completes.