2019-10-22 03:12 CEST

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0006302Spring engineGeneralpublic2019-10-04 14:23
Reporteraeonios 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusfeedbackResolutionopen 
Product Version 
Target VersionFixed in Version 
Summary0006302: Convert Spring to use a local JRE for java AIs
DescriptionAsking people to install the correct (32/64 bit) version of java on their system in order to run java AIs is completely impractical. It's also not possible/practical to install system-level java through steam.

Oracles specifically states that you're allowed to package the JRE with your application pretty much however you like as long as you don't tamper with the JRE's contents. (https://java.com/en/download/faq/distribution.xml)

The easiest way to deal with this is simply to package the correct JRE into spring's directory and have it run the JRE's from there rather than going to the system. That way we can just dump the appropriate JRE (or all of the possibly used ones) into (spring-root)/java/JRE-x as is and distribute it via steam or rapid or whatever.
Additional InformationEnd java AI discrimination. :P
Tagscompatibility
Checked infolog.txt for ErrorsIrrelevant
Attached Files

-Relationships
+Relationships

-Notes

~0020167

Anarchid (reporter)

Passing a JAVA_HOME env-var to spring overrides any other methods of detecting the JRE location, so presumably e.g. ZK could distribute its own JRE alongside a Java AI and just launch Spring with this env var.

~0020169

abma (administrator)

would a spring config setting help?

i guess multiple vms are not possible, i.e. a dedicated vm for java ais.

still zk would need to package it, but IMHO a config setting is more practical than an env var i guess?

~0020170

abma (administrator)

oh, sth. like that seems to already exist:

https://github.com/spring/spring/blob/develop/AI/Interfaces/Java/src/main/native/JvmLocater_common.c#L36

jre.path = ... in jre-location.txt should work.

~0020171

abma (administrator)

so no need to "convert": it seems to already support that feature.

can you test it and provide feedback please?
+Notes

-Issue History
Date Modified Username Field Change
2019-09-12 08:12 aeonios New Issue
2019-09-12 08:12 aeonios Tag Attached: compatibility
2019-10-02 18:01 Anarchid Note Added: 0020167
2019-10-04 14:17 abma Note Added: 0020169
2019-10-04 14:22 abma Note Added: 0020170
2019-10-04 14:23 abma Status new => feedback
2019-10-04 14:23 abma Note Added: 0020171
+Issue History