Skip to content

Ignore missing exts on JRuby#9082

Closed
headius wants to merge 1 commit into
jruby:masterfrom
headius:ignore_missing_exts
Closed

Ignore missing exts on JRuby#9082
headius wants to merge 1 commit into
jruby:masterfrom
headius:ignore_missing_exts

Conversation

@headius

@headius headius commented Nov 19, 2025

Copy link
Copy Markdown
Member

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

headius commented Nov 19, 2025

Copy link
Copy Markdown
Member Author

@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
Copy Markdown
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

eregon commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

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

@eregon

eregon commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

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

headius commented Dec 10, 2025

Copy link
Copy Markdown
Member Author

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