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
cleanup: fix portable Ruby behaviour. #9479
cleanup: fix portable Ruby behaviour. #9479
Conversation
Review period will end on 2020-12-10 at 11:34:29 UTC. |
Library/Homebrew/brew.sh
Outdated
@@ -349,7 +349,7 @@ then | |||
else | |||
# Used in ruby.sh. | |||
# shellcheck disable=SC2034 | |||
HOMEBREW_MACOS_SYSTEM_RUBY_NEW_ENOUGH="1" | |||
export HOMEBREW_MACOS_SYSTEM_RUBY_NEW_ENOUGH="1" |
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.
I wonder if we should export it in the Declared in brew.sh
section below for consistency (that is, set it here, but export it there) and so that there is only one place to keep track of all exported variables.
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.
@maxim-belkin Good idea.
I also think, ideally, that all this cleanup.rb
logic would go away and it'd just shell out to ruby.sh
or similar instead so we can consolidate this logic in one place.
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.
(@maxim-belkin if you'd be able to pick that up I'd be very grateful ❤️)
Better match the macOS behaviour in `ruby.sh`. Fixes #9469
f4abf54
to
adab0fb
Compare
Review period ended. |
Better match the macOS behaviour in
ruby.sh
.Fixes #9469