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
Don't install platform specific gems on truffleruby #4333
Conversation
8d8290f
to
f49fca1
Compare
It's more straightforward.
Truffleruby lazily loads rubygems. Avoid that so that our reexec logic works there too.
f49fca1
to
acd7e4e
Compare
04f5352
to
acd7e4e
Compare
It should be ready now! I added a bare truffleruby bundler CI to make sure the issue is caught: https://github.com/rubygems/rubygems/pull/4333/checks?check_run_id=1822918747. |
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 looks great, thank you for the fix!
It'd be great to merge this and have a release, we're seeing more reports of this issue (e.g., oracle/truffleruby#2246) |
I'll release this at the beginning of next week. |
…deps Don't install platform specific gems on truffleruby (cherry picked from commit d7941db)
What was the end-user or developer problem that led to this PR?
Sometimes platform specific specs are installed on truffleruby.
What is your fix for the problem, implemented in this PR?
Handle the case where an implementation registers itself as a "pure ruby platform" by setting
Gem.platforms == [Gem::Platform::RUBY]
.Fixes #4330.