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

test-extended failing #1380

Closed
BanzaiMan opened this Issue Jan 5, 2014 · 2 comments

Comments

Projects
None yet
3 participants
@BanzaiMan
Copy link
Member

BanzaiMan commented Jan 5, 2014

WEBrick tests are failing at the moment.

https://travis-ci.org/jruby/jruby/jobs/16401033#L287

From the stack trace, you see that webrick/config.rb:14 is requiring webrick/utils, but we are picking up the one in the test tree (https://github.com/jruby/jruby/blob/e86a6b53c016e6c267865fdb49b93bdc729030d3/test/mri/webrick/utils.rb) instead of the one in the lib tree.

I suspect that f98f362 added tests that exposes this issue.

@headius

This comment has been minimized.

Copy link
Member

headius commented Jan 5, 2014

I poked around MRI's test run, and it appears they explicitly pass -I pointing at lib and . before going on to run the tests. This is probably the difference; we just rely on them being in load path by default, but load path is searched after -I paths on the command line.

Testing this theory now.

@headius

This comment has been minimized.

Copy link
Member

headius commented Jan 5, 2014

That appears to be working locally. I'll make sure I have a good local test:mri run and then push.

@headius headius closed this in 26a7f4c Jan 5, 2014

@enebo enebo modified the milestones: JRuby 1.7.10, JRuby 1.7.11 Feb 24, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.