Skip to content

Remove extensions and fully deprecate the gem - #79

Open
headius wants to merge 1 commit into
ruby:masterfrom
headius:empty_gem
Open

Remove extensions and fully deprecate the gem#79
headius wants to merge 1 commit into
ruby:masterfrom
headius:empty_gem

Conversation

@headius

@headius headius commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

The functionality of this gem was moved into core Ruby starting with Ruby 3.2, and at this point both the C extension and the Java extension define no additional classes or methods. This patch fully removes both extensions, replacing them with a single io/wait.rb that warns about the gem deprecation. All other files have been updated to reflect the removed extensions an deprecation.

Fixes #78

@headius

headius commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

@nobu @hsbt The "empty Java extension" version of the gem could be released now (#77), but if this were merged we could simply do a "final" release that fully deprecates the gem.

cc @sampokuokkanen

The functionality of this gem was moved into core Ruby starting
with Ruby 3.2, and at this point both the C extension and the Java
extension define no additional classes or methods. This patch
fully removes both extensions, replacing them with a single
io/wait.rb that warns about the gem deprecation. All other files
have been updated to reflect the removed extensions an deprecation.

Fixes ruby#78
@headius

headius commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

I did not remove the tests, which causes this PR to fail: jruby/jruby#9656 needs to be merged and released to pass the tests, but it needs a release of io-wait before it can be merged.

We probably could just remove the tests here too.

@headius

headius commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

needs to be merged and released

Ok, perhaps not. Seems like it passed ok in CI.

@nobu

nobu commented Sep 11, 2026

Copy link
Copy Markdown
Member

needs to be merged and released

Ok, perhaps not. Seems like it passed ok in CI.

Now the tests also can be removed?

nobu added a commit to nobu/ruby that referenced this pull request Sep 11, 2026
nobu added a commit to nobu/ruby that referenced this pull request Sep 11, 2026
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.

Fully zero out the extension

2 participants