Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Install fails on jruby-1.6.7.2-d19 #21

Closed
simeonwillbanks opened this Issue · 5 comments

3 participants

Simeon Willbanks Ezekiel Templin Peter Ohler
Simeon Willbanks

The full details are on Travis:
http://travis-ci.org/#!/simeonwillbanks/nerd_quiz/jobs/1525709

Thanks for Oj!

Simeon

Ezekiel Templin
ezkl commented

See Travis' docs regarding JRuby and C extensions:

JRuby: C extensions support is disabled

Please note that C extensions are disabled for JRuby on travis-ci.org. The reason for doing so is to bring it to developers attention that their project may have dependencies that should not be used on JRuby in production. Using C extensions on JRuby is technically possible but is not a good idea performance and stability-wise and we believe continuous integration services like Travis should highlight it.

So if you want to run CI against JRuby, please check that your Gemfile takes JRuby into account. Most of popular C extensions these days also have Java implementations (jsom gem, nokogiri, eventmachine, bson gem) or Java alternatives (like JDBC-based drivers for MySQL, PostgreSQL and so on).

Simeon Willbanks

"Please note that C extensions are disabled for JRuby on travis-ci.org. The reason for doing so is to bring it to developers attention that their project may have dependencies that should not be used on JRuby in production."

Should Oj's documentation mention that JRuby support requires C extensions to be enabled?

I don't see this mentioned within the repository.

$ git remote show origin

$ ack -i jruby *

README.md
60:Oj is compatible with Ruby 1.8.7, 1.9.2, 1.9.3, JRuby, RBX, and the latest 2.0dev.

build_test.sh
7: jruby-1.6.7\

ext/oj/fast.c
365:#ifdef JRUBY_RUBY

ext/oj/oj.c
313:#ifndef JRUBY_RUBY
314: // JRuby gets confused with what is the real fileno.

test/test_mimic.rb
75: if 'rubinius' == $ruby || 'jruby' == $ruby || '1.8.7' == RUBY_VERSION
(END)

Peter Ohler
Owner

I added a comment in the README.md to note that JRuby must be built with extensions enabled.

Peter Ohler ohler55 closed this
Simeon Willbanks

@ohler55 Thanks so much and I really appreciate all your hard work with this gem!

Peter Ohler
Owner

Always nice to hear, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.