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

Stub gemspec for JRuby #598

Merged
merged 1 commit into from
Feb 21, 2023
Merged

Stub gemspec for JRuby #598

merged 1 commit into from
Feb 21, 2023

Conversation

headius
Copy link
Contributor

@headius headius commented Feb 8, 2023

JRuby has its own implementation of the openssl library in jruby-openssl. The simplest way for us to allow users to set openssl as a gem dependency is to ship a stub gem that just depends on jruby-openssl. This patch adds that to the gemspec. Additional work may be required to fit this stub gem into the test and release process.

See #20 for more details.

JRuby has its own implementation of the `openssl` library in
jruby-openssl. The simplest way for us to allow users to set
openssl as a gem dependency is to ship a stub gem that just
depends on jruby-openssl. This patch adds that to the gemspec.
Additional work may be required to fit this stub gem into the test
and release process.

See ruby#20 for more details.
@headius headius mentioned this pull request Feb 8, 2023
@hsbt
Copy link
Member

hsbt commented Feb 9, 2023

@rhenium Do you have any concerns?

@kares
Copy link

kares commented Feb 9, 2023

👍 believe this would be usable as an interim compromise solution, for JRuby users, but
requires maintainers to keep pushing the openssl-java variant on releases ...

@headius
Copy link
Contributor Author

headius commented Feb 9, 2023

keep pushing the openssl-java variant

If we moved jruby-openssl guts into here it would be the same way. There's just no other way to handle this since RubyGems does not support any way to have "alternate" implementations for a given gem name.

@hsbt
Copy link
Member

hsbt commented Feb 10, 2023

@headius This PR has a merge period? If there is no response from @rhenium for its period, I'll merge this before your request time

@hsbt hsbt merged commit b5ef6d1 into ruby:master Feb 21, 2023
@headius headius deleted the jruby_openssl branch February 24, 2023 04:56
@headius
Copy link
Contributor Author

headius commented Feb 24, 2023

Thank you for the merge! I have been under the weather this week, but I look forward to seeing a release of the openssl-java gem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants