Skip to content
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

Skip extension builds for default gems #8415

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

headius
Copy link
Member

@headius headius commented Nov 9, 2024

Seems to be issues launching these subprocesses via Maven. While we investigate that, temporarily disable extension building for default gems.

The problem manifests during the default gem install of the fiddle gem, which generates a dummy Makefile on JRuby because it uses the FFI version instead. The launch of extconf.rb exhibits problems in various environments:

  • In CI, it ends up running with the system default Java rather than the one configured in JAVA_HOME.
  • On local Linux environments, it fails to launch due to a "bad file descriptor: /bin/sh" error.

It seems that the subprocess launch is losing environment in some broken way.

@headius headius added this to the JRuby 9.4.10.0 milestone Nov 9, 2024
Seems to be issues launching these subprocesses via Maven. While
we investigate that, temporarily disable extension building for
default gems.

The problem manifests during the default gem install of the fiddle
gem, which generates a dummy Makefile on JRuby because it uses the
FFI version instead. The launch of extconf.rb exhibits problems in
various environments:

* In CI, it ends up running with the system default Java rather
  than the one configured in JAVA_HOME.
* On local Linux environments, it fails to launch due to a "bar
  file descriptor: /bin/sh" error.

It seems that the subprocess launch is losing environment in some
broken way.
@headius headius force-pushed the skip_default_gem_extensions branch from d4849c5 to 280b0f8 Compare November 9, 2024 20:39
@headius
Copy link
Member Author

headius commented Nov 11, 2024

This appears to work fine but @enebo pointed out we will eventually need this to start building exts again for the prism gem. For now, this will allow 9.4 (master) and 10 to build without running into the env issues.

@headius headius merged commit 6f9df83 into jruby:master Nov 11, 2024
93 of 95 checks passed
@headius headius deleted the skip_default_gem_extensions branch November 11, 2024 19:11
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.

1 participant