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

Remove remaining "19" methods #8149

Open
wants to merge 2 commits into
base: 9.5-dev
Choose a base branch
from

Conversation

headius
Copy link
Member

@headius headius commented Mar 12, 2024

These methods were not deprecated before JRuby 10, so they were left in place for one additional "major" release. This deletes them.

They are very low risk of causing compile or execution problems, but this safer approach gives any remaining users a full version cycle to switch to the long-available normal versions.

See #8143 for the original pass that removed already- deprecated methods.

@headius headius added this to the JRuby 9.6.0.0 milestone Mar 12, 2024
@headius headius marked this pull request as ready for review March 12, 2024 04:19
@headius headius force-pushed the remove_remaining_version_methods branch from 8e69949 to 4553e9c Compare March 12, 2024 04:52
@headius
Copy link
Member Author

headius commented Mar 12, 2024

A wrinkle here is the use of RubyString#cat19 from StringIO (ruby/stringio#83). The replacement names were originally added to 9.5/10 in #8143 and backported to 9.4 in 4114fc5.
For purposes of this PR, we will not remove the methods. Instead, we will track this migration in ruby/stringio#83.

@headius
Copy link
Member Author

headius commented Mar 12, 2024

I modified the patch in ruby/stringio#84 to work on both old and new methods, so once that is released we can update this PR to use the new stringio version (or just update 9.4 to that version and merge it forward).

@headius headius force-pushed the remove_remaining_version_methods branch 2 times, most recently from 989ade7 to 9529d78 Compare March 13, 2024 15:09
These methods were not deprecated before JRuby 10, so they were
left in place for one additional "major" release. This deletes
them.

They are very low risk of causing compile or execution problems,
but this safer approach gives any remaining users a full version
cycle to switch to the long-available normal versions.

See jruby#8143 for the original pass that removed already-
deprecated methods.
@headius headius force-pushed the remove_remaining_version_methods branch from 9529d78 to 725379c Compare March 13, 2024 17:11
@headius
Copy link
Member Author

headius commented Mar 13, 2024

This has been rebased on a merge of master to 9.5-dev and goes back to removing all these methods. The stringio issue will be resolved once a release contains ruby/stringio#84 and ruby/stringio#88, which lets it support both old and new names. We will ship 9.4 with both old and new names, and 9.5 with new names only.

headius added a commit to headius/json that referenced this pull request Mar 26, 2024
These methods have been deprecated for some time and we would like
to delete them.

See jruby/jruby#9143 and jruby/jruby#8149.
headius added a commit to headius/json that referenced this pull request Mar 26, 2024
These methods have been deprecated for some time and we would like
to delete them.

See jruby/jruby#9143 and jruby/jruby#8149.
@headius
Copy link
Member Author

headius commented Mar 26, 2024

A few more method references were found in json: flori/json#576

Once released, that PR will move to versions that exist in JRuby 9.4 so no special gymnastics are required (but we'll need that json released before we can make this change safely).

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.

None yet

1 participant