-
-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Disable more deprecations. #4392
Disable more deprecations. #4392
Conversation
@@ -363,15 +349,6 @@ def make_relative_symlink(src) | |||
File.symlink(src.relative_path_from(dirname), self) | |||
end | |||
|
|||
unless method_defined?(:/) | |||
def /(other) |
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.
Where did this go?
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.
Ruby 2.3.3 always defines this method.
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.
Nice!
@@ -95,7 +95,6 @@ def parse | |||
next if encoding.nil? | |||
|
|||
# use the specified encoding for the rest of the file | |||
next unless String.method_defined?(:force_encoding) |
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.
Don't change vendored code.
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.
Agreed.
Library/Homebrew/extend/fileutils.rb
Outdated
# @private | ||
alias old_ruby ruby | ||
end | ||
|
||
# Run the `ruby` Homebrew is using rather than whatever is in the `PATH`. | ||
def ruby(*args) |
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.
Seems like this isn't used anywhere. Should we deprecate this?
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.
Good idea!
- Turn more old `odeprecated` into `odisabled` - Remove more old `odisabled` - Remove code for older Ruby versions
These don't need to live on FileUtils and don't really make sense there either.
odeprecated
intoodisabled
odisabled
This should not be merged until we are sure the next version will be 1.7.0.
CC cask folks for the @Homebrew/cask changes.
brew style
with your changes locally?brew tests
with your changes locally?