Fix library loading on android

[Berstanio]

On android we load natives with

System.loadLibrary(platformName);

which expects unprocessed libname:

Loads the native library specified by the libname argument. The libname argument must not contain any platform specific prefix, file extension or path.

[SimonIT]

Good catch, thanks

[obigu]

Interesting, might this be related? libgdx/libgdx#5863

[Berstanio]

Interesting, might this be related? libgdx/libgdx#5863

No, this fixes a complete breakage of android lib loading introduced in jnigen 2.5.0. 2.5.0 isn't used in libGDX yet

[SimonIT]

Hmm, after having a second look, are you sure it's needed? Because these methods

gdx-jnigen/gdx-jnigen-loader/src/main/java/com/badlogic/gdx/utils/Os.java

Lines 7 to 26 in 21a622d

	public String getJniPlatform () {
	if (this == Os.Windows) return "win32";
	if (this == Os.Linux) return "linux";
	if (this == Os.MacOsX) return "mac";
	return "";
	}
	public String getLibPrefix () {
	if (this == Os.Linux || this == Os.Android || this == Os.MacOsX) {
	return "lib";
	}
	return "";
	}
	public String getLibExtension () {
	if (this == Os.Windows) return "dll";
	if (this == Os.Linux) return "so";
	if (this == Os.MacOsX) return "dylib";
	return "";
	}

handle the case for android, I think. Or is the lib added by getLibPrefix wrong?

[Berstanio]

Yeah, the lib added is wrong and the suffix is wrong. It will currently result in trying to load something like:
liblib<name>..so

[SimonIT]

Ah, I added it probably because the file has added lib to the name, but for loading it's not needed.
https://github.com/SimonIT/gdx-jnigen/blob/9d0e3214a1c64cb1f9e1d16a4399c23e5db742b0/gdx-jnigen/src/main/java/com/badlogic/gdx/jnigen/BuildTarget.java#L157-L159