build(distribution): re-enable SecurityManager in distributions running under Java 18 and above #5091
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contains
When running under Java 18/19, the game crashes on start-up when attempting to set a
SecurityManager
.SecurityManager
was deprecated in Java 17 and is disabled by-default from Java 18.This fixes #5090 by re-enabling
SecurityManager
in the launch scripts. The changes should only affect users running Java 18 and later.How to test
gradlew distZip
.(I tested this with the one from https://adoptium.net)
JAVA_HOME
environment variable to point at the Java 19 distribution../Terasology
(orTerasology.bat
) from the built distribution folder.Notes