Fix for #4147 #4153

Merged
merged 1 commit into from Sep 14, 2016

Projects

None yet

3 participants

@smk0621
Contributor
smk0621 commented Sep 14, 2016 edited

When check the OS platform, the RUBY_PLATFORM return 'java' in JRuby.
It is better to use RbConfig::CONFIG['host_os'] .

fixes #4147

@smk0621 smk0621 Fix JRuby issue#4147
When check the OS platform, the RUBY_PLATFORM return 'java' in JRuby.
2f9ec95
@kares kares added this to the JRuby 9.1.6.0 milestone Sep 14, 2016
@kares kares merged commit 6ce5e55 into jruby:master Sep 14, 2016

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@enebo
Member
enebo commented Sep 14, 2016

@headius can you commit this upstream? I don't know if they want rbconfig but this is a better check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment