Panic raised when jvm.dll not found

rage / tmc-langs-rust

A library and CLI for supporting different programming languages in the TestMyCode programming assignment evaluator.

https://tmc.mooc.fi/

Apache License 2.0

7 stars 9 forks source link

Panic raised when jvm.dll not found #163

Closed ShootingStar91 closed 3 years ago

ShootingStar91 commented 3 years ago

Crate j4rs panics on this line if JAVA_HOME environment variable is not set correctly or jvm.dll is not found from the path set as JAVA_HOME.

thread 'main' panicked at 'Could find the jvm dynamic library: JavaLocatorError { description: "Java is not installed or not added in the system PATH" }', C:\Users\Arttu\.cargo\registry\src\github.com-1ecc6299db9ec823\j4rs-0.13.0\src\api_tweaks\generic.rs:41:38

tmc-langs-rust calls it on plugins/java/src/lib.rs line 115

Heliozoa commented 3 years ago

Thanks, this was previously taken care of by catching it in the langs CLI but I forgot to reconsider it when langs started being used as a library as well.