Bug #18600
closedAliased method visibility issue on Ruby 3.1
Description
It seems methods on classes that were originally private in the parent and made public in the child can no longer be aliased correctly:
class C
public :system
alias_method :bar, :system
alias_method :system, :bar
end
p RUBY_VERSION
p [C.new.respond_to?(:system), C.new.respond_to?(:system, true)]
On Ruby <= 3.0.3, this returns [true, true]
, but on Ruby >= 3.1.0 this returns [false, true]
. The aliased method seems to revert visibility to the parent.
Updated by gmcgibbon (Gannon McGibbon) almost 3 years ago
May be related to https://github.com/ruby/ruby/pull/4149.
Updated by byroot (Jean Boussier) almost 3 years ago
- Backport changed from 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN, 3.1: UNKNOWN to 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: UNKNOWN
Updated by jeremyevans0 (Jeremy Evans) almost 3 years ago
Thanks for the report. I bisected this to 71c746379d5872e250d90ae45c585760afaf9516 (which looks like it matches the 4149 PR). That commit was done for performance, so we probably don't want to just revert it. It probably needs a tweak to make sure the visibility is set correctly in this corner case.
Updated by jeremyevans0 (Jeremy Evans) almost 3 years ago
- Backport changed from 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: UNKNOWN to 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: REQUIRED
Updated by jeremyevans0 (Jeremy Evans) almost 3 years ago
I've submitted a pull request to fix this: https://github.com/ruby/ruby/pull/5592
Updated by jeremyevans (Jeremy Evans) almost 3 years ago
- Status changed from Open to Closed
Applied in changeset git|0c6e24d102e894a7211a596e6aa95828b1cf4406.
Fix visibility of alias of zsuper methods
This was broken by 71c746379d5872e250d90ae45c585760afaf9516.
Fixes [Bug #18600]
Updated by nagachika (Tomoyuki Chikanaga) over 2 years ago
- Backport changed from 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: REQUIRED to 2.6: DONTNEED, 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONE
ruby_3_1 9d11ddcf6ace6676cea7cac836b4302240e3efb4 merged revision(s) 0c6e24d102e894a7211a596e6aa95828b1cf4406.