Skip to content
This repository was archived by the owner on Sep 15, 2026. It is now read-only.

Remove extensions and fully deprecate the gem - #79

Merged
nobu merged 1 commit into
ruby:masterfrom
headius:empty_gem
Sep 13, 2026
Merged

nobu merged 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
@headius

headius commented Sep 12, 2026

Copy link
Copy Markdown
Contributor Author

Yes, tests can be removed as well if they are now based in ruby/ruby. We can strip this down to nothing.

@nobu
nobu merged commit 4ae06a4 into ruby:master Sep 13, 2026
18 checks passed
nobu added a commit to nobu/ruby that referenced this pull request Sep 13, 2026
nobu added a commit to nobu/ruby that referenced this pull request Sep 13, 2026
@headius
headius deleted the empty_gem branch September 14, 2026 13:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fully zero out the extension

2 participants