-
Notifications
You must be signed in to change notification settings - Fork 92
require jruby explicitly #132
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
Conversation
|
@headius Can you review 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.
This change is safe and works on all JRuby versions. From JRuby 9.3 forward this require is needed to expose some JRuby-specific utility functions.
|
We have now another error. Is this related? |
JRuby previously only shipped the extension and We would prefer to source those files from the gem, but that will require @hsbt or someone else responsible for the racc gem to do a release. |
|
@headius I'm going to release the current master branch as racc-1.5.0 after cleaning-up the some of obsoleted files. |
|
@hsbt Missed this until now. Thank you! |
fixes jruby/jruby#6125
related to jruby/jruby#5381