-
Notifications
You must be signed in to change notification settings - Fork 220
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
Java7CallGraphTest.testOcamlHelloHash() failing #60
Comments
@cos if you have time to see if you get this failure, that would be great. |
I have 1.7.0_71, and it seems to work. I'll try to get _72 and check...
From: Manu Sridharan notifications@github.com To: wala/WALA WALA@noreply.github.com Cc: Julian Dolby/Watson/IBM@IBMUS Date: 11/11/2014 02:11 PM Subject: [WALA] Java7CallGraphTest.testOcamlHelloHash() failing (#60) This regression test fails for me when running on JDK 1.7.0_72. I wonder if — |
I have Java 8 on my system so it is a bit harder to test locally. Still, the #59 pull request has passed on JDK 1.7.0_72: The exception is: The exception is thrown here: WALA/com.ibm.wala.core.tests/src/com/ibm/wala/core/tests/shrike/DynamicCallGraphTestBase.java Line 99 in f4f6f06
It means the resources from the There's also an open class loader there. I'll try closing it and see how it goes. |
Ugh. I would think that each run on Travis boots a fresh instance? I https://travis-ci.org/wala/WALA/builds/40020629 (note the output is from Anyway, I was unable to reproduce locally, and I really don't understand On Tue, Nov 11, 2014 at 4:38 PM, Cosmin Radoi notifications@github.com
|
@juliandolby, did you fix this one? It looks like commit 71f16d4 is related |
I don't know... hard to tell because this bug only happens sometimes From: Manu Sridharan notifications@github.com To: wala/WALA WALA@noreply.github.com Cc: Julian Dolby/Watson/IBM@IBMUS Date: 11/11/2014 09:33 PM Subject: Re: [WALA] Java7CallGraphTest.testOcamlHelloHash() failing (#60) Julian, did you fix this one? It looks like commit 71f16d4 is related — |
Closing as this seems to be working on Travis now. |
This regression test fails for me when running on JDK 1.7.0_72. I wonder if the JDK update caused the problem? @juliandolby can you reproduce?
The text was updated successfully, but these errors were encountered: