search

plasma-umass / doppio

Breaks the browser language barrier (includes a plugin-free JVM).
http://plasma-umass.github.io/doppio-demo
MIT License
2.16k stars 175 forks source link

Class loading errors #244

Closed lavelle closed 11 years ago

lavelle commented 11 years ago

Just realised a minor flaw in my testing strategy for the Bower integration – I didn't actually try and run any Java programs. Doing so now just lands me a java/lang/ClassNotFoundException, specifically

Uncaught Error: Error in get_loaded_class: Class Ljava/lang/ClassNotFoundException; is not loaded. ClassLoader.coffee:208
ClassLoader.get_loaded_class ClassLoader.coffee:208
ClassLoader.get_resolved_class ClassLoader.coffee:214
ClassLoader.get_initialized_class ClassLoader.coffee:221
(anonymous function) ClassLoader.coffee:473
(anonymous function) jvm.coffee:149
(anonymous function) ClassLoader.coffee:447
(anonymous function) jvm.coffee:73
(anonymous function) browserfs.js:1803
handleMessage browserfs.js:3133

tools/preload is generated correctly, and mini-rt.tar is definitely getting loaded.

Is this some change brought upon by the BrowserFS integration? I've seen that the file structure has changed, currently I'm running cd ../sys and then java classes/foo/Bar every time the page loads.

perimosocordiae commented 11 years ago

That's strange; I was able to run programs fine when I tested. (Both before and after Bower integration.)

I agree that we need to fix up some things for the demo, now that the file structure is different.

lavelle commented 11 years ago

Hmm, well this is on my Swing branch, maybe it's some change I've made. I'll check if it works on master.

lavelle commented 11 years ago

Nope, no dice on master either.

perimosocordiae commented 11 years ago

I can't reproduce. I just checked out a clean repo, ran make release, then ran cd ../sys and java classes/demo/Fib. Everything worked as expected.

lavelle commented 11 years ago

Oh right, I was using dev. Just tried release and I get the same error though. I don't have a fresh clone but I've pulled in the latest master so it should be the same.

perimosocordiae commented 11 years ago

Try the usual suspects: make clean, use an incognito window (or a different browser entirely), etc.

lavelle commented 11 years ago

Same problem :/

jvilk commented 11 years ago

This is only present in Giles' fork, since he had custom option parsing logic that didn't merge in our path changes from the BFS changes.

I fixed it: https://github.com/lavelle/doppio/commit/137ee31820fd560c308fce6960e2f423b68ea779