-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Fix issues introduced by latest Java additions #357
Conversation
fyrz
commented
Oct 21, 2014
- Makefile did not cleanup after test
- Crossbuild sets explicit JDK6 which is wrong because Java7 dependencies were introduced lately.
@yhchiang can you please look at this. |
I see that the Travis-CI build is failing because of the test suite. This seems to be an intermittent problem, sometimes (but not always) when I run the test suite locally it also fails. I don't know if this is a test suite problem or an underlying Rocks issue. However I would suggest this should be fixed as a priority, if we can't rely on the test suite, well that is a a big problem. |
@adamretter We will fix it ASAP |
Fixed -- 90f1564 |
@adamretter @fyrz can you guys add java test to travis build? |
@igorcanadi Thanks for the fix, I have sent a pull-request for Travis. |
@igorcanadi can you merge this one also it solves some issues with the last additions. |
@adamretter can you please review @fyrz's patch? :) |
@fyrz: thanks for the patch. The diff looks good to me. |
Fix issues introduced by latest Java additions
bump dependencies