godot-kotlin-jvm icon indicating copy to clipboard operation
godot-kotlin-jvm copied to clipboard

Make jre folder platform dependent

Open chippmann opened this issue 1 year ago • 3 comments

This makes the jre folder platform dependent.

This makes it possible for a project to have multiple jre folder simultaneously on the file system. The module then automatically picks the correct one for the current platform. The main advantage of this is that on a project with multiple people working on it, the embedded jre can be committed to the repository for each platform together with the built jar and a project member does not need to have java locally installed and JAVA_HOME set. Which is especially useful if one is working with non programmers.

Note: Breaking change. Needs special notes in release notes!

chippmann avatar May 12 '24 15:05 chippmann

I tested linux editor and export templates. Could you test windows and mac? (I don't think testing on mobile is necessary but you can if you want to :-) )

chippmann avatar May 18 '24 11:05 chippmann

Tested the exports and editors for linux amd64 with the changes @CedNaru requested. Will test on macos later today

@piiertho need a review again from you as these were quite significant changes again

chippmann avatar May 26 '24 10:05 chippmann

Tested editor and export on macOS as well.

If you @CedNaru or @piiertho could test on windows, that would be appreciated.

chippmann avatar May 26 '24 11:05 chippmann

Tested it on mac and windows as well in the meantime. Just graalvm is missing testing on those platforms

chippmann avatar Jul 25 '24 14:07 chippmann

I don't remember: did we test it on all platforms ?

piiertho avatar Aug 21 '24 15:08 piiertho

I did on all platforms except mobile and graalvm for windows and macos Although it might be worth to test again after my last change even though i did not change much about the logic of the export

chippmann avatar Aug 21 '24 15:08 chippmann

After my last change i did test the exports on macos (as thats where I'm working on today) I'm currently in the process of trying to add export tests to the ci/cd pipeline. But depending on how that goes, it might be faster to test locally again.

chippmann avatar Aug 21 '24 15:08 chippmann

I will test it on graalVM native-image for iOS.

piiertho avatar Aug 26 '24 08:08 piiertho

Tested on iOS

piiertho avatar Aug 31 '24 09:08 piiertho