Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Conversation

@headius
Copy link
Member

@headius headius commented Nov 19, 2025

This problem does not seem to be going away, so ignore all missing extensions on JRuby for now.

See ruby/rubygems#3520

This problem does not seem to be going away, so ignore all missing
extensions on JRuby for now.

See ruby/rubygems#3520
@headius headius added this to the JRuby 10.0.3.0 milestone Nov 19, 2025
@headius
Copy link
Member Author

headius commented Nov 19, 2025

@deivid-rodriguez @eregon I'm going ahead with a PR to patch this method in JRuby. I'd prefer not to have to wipe out the extension detection, but it's too specific and usually erroneous in JRuby, which has very few extension ABI issues.

@deivid-rodriguez
Copy link
Contributor

Unfortunately I'm no longer involved in the development of RubyGems, so I can't help here. Please check out with the new maintainers team and I appreciate if you don't ping me directly with respect to RubyGems related issues (no big deal, but I'm no longer the right person to ping here).

Good luck!

@eregon
Copy link
Member

eregon commented Nov 20, 2025

This sounds like the wrong fix to me, see ruby/rubygems#3520 (comment)

@eregon
Copy link
Member

eregon commented Nov 20, 2025

An example problem with this hack is if there is a gem which does something with gemspec.extensions = on JRuby then you would get a pretty late error vs a clearer error that somehow the extension didn't get built, e.g. when reusing a GEM_HOME from another JRuby version or so.
gemspec.extensions = is not just for C extensions, it can run an arbitrary Ruby script, and it is used like that for some gems.

@headius
Copy link
Member Author

headius commented Dec 10, 2025

Closing in favor of #9107.

@headius headius closed this Dec 10, 2025
@headius headius deleted the ignore_missing_exts branch December 10, 2025 07:38
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.

3 participants