-
Notifications
You must be signed in to change notification settings - Fork 18
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
Gem fails to install under JRuby #75
Comments
This can be changed by trocla removing the openssl dependency (or releasing a -java version of the gem that uses jruby-openssl), or it will have to wait until we can get the openssl gem maintainer to release a no-op version for JRuby that installs jruby-openssl. |
I am just a bit confused how that slipped in. It did when I updated the gemspec, but I wonder why.... |
Ah it came in through #69 - that explains it. |
So it looks like we do not need the OpenSSL gem at all. BUT I tried to make sure we don't regress into a broken jruby variant (since it is important for puppet-server) by re-adding jruby to the CI: #76 BUT it fails on jruby as all code that relates to Jruby CI runs fine locally though. I have a simple reproducer that shows that See here: https://github.com/duritong/ruby-actions-test/actions/runs/3859056296 @headius since we somehow have your attention here: why do we have that issue? I am not sure where to open an issue for that over at jruby, ruby/setup-ruby or github (where???) |
I decided to report it to ruby/setup-ruby as a first try: ruby/setup-ruby#448 |
FWIW Puppet Server's JRuby is currently pegged at 9.3.9.0 (via jruby-utils), while your CI is using nightly snapshot of JRuby. Are you seeing the same issue with that version of JRuby? |
Hi
@jcharaoui did you find a workaround to install trocla 0.5.0 in jruby ? |
Currently the easiest method is to just install trocla 0.4.0... If you're on Debian bookworm/sid, you can install the |
I use actually 0.4.0 and I want feature that are available on 0.5.0 release :( |
Following up here: the SecureRandom issue in 9.4.0.0 is fixed on master and will be released in 9.4.1.0 early next week. The JRuby team is here to help make sure this gem runs green in CI or answer any questions! |
@headius thank you for the infos. The gem is already green on jruby-head, as soon as we have it as well on latest jruby (according to you next week), I'll make a new release that addresses this issue fully. |
@duritong Does that mean the upcoming version of trocla require JRuby 9.4 and above? Because as I mentioned above, Puppet Server currently pegs its integrated JRuby at |
If we can get a version of the openssl gem released for JRuby that just depends on jruby-openssl, it should be installable on all supported JRuby versions. The SecureRandom issue was only in 9.4.0. |
FTR this work appears to be currently tracked in ruby/openssl#20 |
fixed with bf52792 |
so 0.5.1 was released that does not depend on the openssl gem anymore and passes on all supported ruby versions. |
The addition of
openssl
as a dependency is causing the gem to fail to install in the JRuby environment:There's already a
This is because the
openssl
gem will try (and fail) to build its native C extension. However the Gemfile already brings injruby-openssl
so it should work to only add theopenssl
dependency when not using JRuby.The text was updated successfully, but these errors were encountered: