-
Notifications
You must be signed in to change notification settings - Fork 63
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
Jruby Support #72
Comments
Hi @dometto, I'm not sure why it's not supported to be honest, but your experience leads me to believe that it isn't. As far as adding support, I'd need a patch to get it building under CI. Is this something you'd be interested in working on? |
Is it redcarpet even used at all? I fail to find where is being used Update: Removed at #115 |
Looks like there's a bunch of things to investigate for JRuby support: https://github.com/sinatra/mustermann/runs/7393804603?check_suite_focus=true#step:4:1 |
Issue tracking support: sinatra#72
Issue tracking support: sinatra#72
The docs state that
jruby
is currently not supported, but I cangem install mustermann
just fine. Is it ready for use withjruby
after all?I have tried to run the specs on
jruby
, but this fails because theredcarpet
gem required bysupport
is not compatible (it relies onposix-spawn
). If I disable theredcarpet
dependency in the gemspec, the specs run but result in 23 failures. I take it that is not related to removingredcarpet
(since else I should be seeing errors concerning that)?If not that, can you indicate what needs to happen to fully support
jruby
? I would be interested in getting that running so when can upgrade gollum tosinatra v2
.The text was updated successfully, but these errors were encountered: