Skip to content
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

require 'ripper' yields const already init warnings #2904

Closed
cpuguy83 opened this issue May 1, 2015 · 2 comments
Closed

require 'ripper' yields const already init warnings #2904

cpuguy83 opened this issue May 1, 2015 · 2 comments

Comments

@cpuguy83
Copy link

@cpuguy83 cpuguy83 commented May 1, 2015

When trying to require ripper on jruby 9k-pre2 we get these warnings:

/opt/jruby/lib/ruby/stdlib/ripper/core.rb:13: warning: already initialized constant SCANNER_EVENT_TABLE
/opt/jruby/lib/ruby/stdlib/ripper/core.rb:13: warning: already initialized constant PARSER_EVENT_TABLE
/opt/jruby/lib/ruby/stdlib/ripper/core.rb:13: warning: already initialized constant Version
/opt/jruby/lib/ruby/stdlib/ripper/core.rb:13: warning: already initialized constant SCRIPT_LINES__

Looking at the referenced core.rb, the only thing I can see is that before doing anything it's requiring ripper.jar... so I'm assuming that these constants are being defined in there.

@enebo
Copy link
Member

@enebo enebo commented May 1, 2015

Hmmm, apparently I added the right mojo in the ripper.jar for it to load itself. @headius can you commit this ripper.rb change to MRI upstream?

@headius
Copy link
Member

@headius headius commented May 5, 2015

Updated our stdlib in jruby/ruby@a0951c6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants