-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Deps: update JRuby to 9.2.18.0 #12791
Conversation
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.
The change itself looks fine, but I would personally rather wait for 9.2.18.0 which will include the resolution for jruby/jruby#6638 WRT spaces in $JAVA_HOME
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.
LGTM, please backport to 7.x
+ Fix: a missing require in bootstrap (cherry picked from commit ee6038a)
Version bump of JRuby.
Release notes
Includes upstream fixes for ARM64 support.
What does this PR do?
Dependency update, release notes (since 9.2.16.0):
https://www.jruby.org/2021/03/29/jruby-9-2-17-0.html
https://www.jruby.org/2021/06/08/jruby-9-2-18-0.html
JRuby 9.2.18.0 include JNR fixes for ARM64: jnr/jnr-ffi#240 jnr/jnr-posix#165
Checklist
Author's Checklist
vendor/jruby/bin/jruby
e.g. frombin/logstash-plugin
(present in 9.2.17.0 but resolved in 9.2.18.0)
another suspicious issue withString#split
Related issues
#12077 (unfortunately the auto-load fixes were not backported to RGs 3.1)