eric / metriks

An experimental library to instrument ruby
MIT License
478 stars 51 forks source link

Make it work on jruby #8

Closed jabley closed 12 years ago

jabley commented 12 years ago

rbtree uses the Ruby C API, meaning that it doesn't play nicely with rubinius and jruby.

eric commented 12 years ago

Thanks for the pull request!

Were you running into problems with running the gem on jruby or just when running the tests?

The gemspec uses the avl_tree gem which is ruby-only. The reason for the rbtree gem in the Gemfile was purely for benchmarking.

I'll look at figuring out a different way to do the benchmarking if this is an issue.

eric commented 12 years ago

Just pushed a fix for this. Let me know if it works for you.

jabley commented 12 years ago

Sorry, just saw this. I was just trying rake test and it was failing.