Skip to content

Check host_os rather than RUBY_PLATFORM#4102

Open
headius wants to merge 2 commits intoruby:mainfrom
headius:wrong_platform_check
Open

Check host_os rather than RUBY_PLATFORM#4102
headius wants to merge 2 commits intoruby:mainfrom
headius:wrong_platform_check

Conversation

@headius
Copy link
Copy Markdown
Contributor

@headius headius commented May 6, 2026

RUBY_PLATFORM has been "java" on JRuby since the mid 2000s, so use "host_os" from RbConfig::CONFIG to compare the host platform.

This fixes installs on FreeBSD and OpenBSD which need to use gmake instead of make.

cc @jeremyevans who ran into this upgrading minitest while running JRuby on OpenBSD. Please confirm!

RUBY_PLATFORM has been "java" on JRuby since the mid 2000s, so use
"host_os" from RbConfig::CONFIG to compare the host platform.

This fixes installs on FreeBSD and OpenBSD which need to use gmake
instead of make.
Comment thread ext/prism/extconf.rb Outdated
Co-authored-by: Jeremy Evans <code@jeremyevans.net>
@headius
Copy link
Copy Markdown
Contributor Author

headius commented May 6, 2026

@jeremyevans Thanks for the catch!

@kddnewton This can be squashed and merged once green.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants