Skip to content

/ jruby

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

Jump to bottom

[ruby 2.4] forwardable.rb is broken on non-MRI #4290

Closed
headius opened this issue Nov 14, 2016 · 2 comments
Closed

[ruby 2.4] forwardable.rb is broken on non-MRI #4290

headius opened this issue Nov 14, 2016 · 2 comments
Labels
JRuby 9000 ruby 2.4 stdlib tests
Milestone
JRuby 9.2.0.0

Comments

@headius
Copy link
Member

@headius headius commented Nov 14, 2016

See https://bugs.ruby-lang.org/issues/12938.

forwardable.rb now unconditionally uses RubyVM features not supported on any other Ruby impl. This prevents any tests that depend on forwardable (at boot) from loading properly.
@headius headius added this to the JRuby 9.2.0.0 milestone Nov 14, 2016
@headius
Copy link
Member Author

@headius headius commented Nov 14, 2016

Ref #4293
headius added a commit that referenced this issue Nov 14, 2016
@headius
Revert to Ruby 2.3.1 forwardable.rb for #4290.
14c4c3d
chrisseaton added a commit to oracle/truffleruby that referenced this issue Oct 18, 2017
@chrisseaton
Use forwardable from 2.3.3
9279865 
jruby/jruby#4290
@chrisseaton chrisseaton mentioned this issue Oct 18, 2017
Merged
chrisseaton added a commit to oracle/truffleruby that referenced this issue Oct 18, 2017
@chrisseaton
Use forwardable from 2.3.3
ad838f5 
jruby/jruby#4290
@headius
Copy link
Member Author

@headius headius commented May 15, 2018

Fixed at some point. Our forwardable matches MRI 2.5 HEAD.
@headius headius closed this May 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
JRuby 9000 ruby 2.4 stdlib tests
Projects
None yet
Milestone
JRuby 9.2.0.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
@headius
You can’t perform that action at this time.