-
Notifications
You must be signed in to change notification settings - Fork 20
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
SyntaxError: rrd.rb:47 during require #2
Comments
Which version of ruby are you using? I've tested here from source and it seems to work on ruby 1.8.7 and 1.9.2.preview. |
jruby 1.4.0 (ruby 1.8.7 patchlevel 174) |
thanks rizzah, i will take a look at it with jruby |
Cool, thanks. Its definitely something within the block for rrd.rb and the wrapper.rb as well. For instance I commented the block out in the rrd.rb, then received the same error in the wrapper.rb. |
rizzah, could you please test it from the source here. i added a small fix and now it seems to work on jruby as well. Can you confirm please? Thanks for the feedback! |
That got it to load. Seeing a new issue when calling functions out of it...
NoMethodError (undefined method |
In my environment the tests did pass. I made some changes, removing the reduce method. Can i have your help again? |
Sure, got this... TypeError (can't convert Hash into String): for what its worth i just changed the reduce methods to inject, which is essentially the same method, received the same error |
Try the latest changes from source, to see if it works. I removed the reduce method, using only each_pair method now. |
TypeError (can't convert Hash into String): |
Graph using jruby is working for me. I can't reproduce this error. I tested using jruby 1.4.0 from rvm. |
hm yup..got it to work. Thanks for the assistance. |
close |
Looks like there is an issue with the rrd.rb file when you try to require it.
require 'rrd.rb'
SyntaxError: rrd.rb:47: , unexpected ','
from rrd.rb:3
from rrd.rb:3:in `require'
from (irb):3
The text was updated successfully, but these errors were encountered: