Classpath separator on linux is different from windows

[elijah629]

On Linux systems, the java classpath separator is a colon instead of a semicolon. I have used conditional compilation to set the separator based on the compilation target.

[John71-Off]

Classpath separator for Windows is ;, for Linux and MacOS :

You should use :

#[cfg(not(target_os = "windows"))] // Linux and MacOS
pub const CLASSPATH_SEP: char = ':';

#[cfg(target_os = "windows")] // Windows
pub const CLASSPATH_SEP: char = ';';

[DeityLamb]

I think it's worth fixing as well

let fixed_lib_path = Path::new(&libraries_path).join(lib_path.replace("/", "\\"));

[John71-Off]

Yes, code should be :

let fixed_lib_path = Path::new(&libraries_path).join(lib_path.replace("\\", "/"));

or

let fixed_lib_path = Path::new(&libraries_path).join(lib_path);

as / is a path separator that works for Linux, MacOS and Windows ;
I don't think that \\ works for OSes other than Windows.