From 3dc53e0351a37fc6d3f6a6e311df0320987edc48 Mon Sep 17 00:00:00 2001
From: Alexander Kriegisch <Alexander@Kriegisch.name>
Date: Thu, 31 Mar 2022 12:50:14 +0700
Subject: POM bug: separate jvm.arg.allowSecurityManager from jvm.arg.addOpens

Before, this was in a single variable, and for JDK 18+, the security
manager setting also falsely overwrote the '--add-opens' command. This
was the root cause for a few dozen LTW tests to fail on JDK 18, if they
were not run in full LTW mode, i.e. in a separate JVM.

After this fix, it should be possible to revert the corresponding
commits, at least their non-cosmetic parts.

Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
---
 run-all-junit-tests/pom.xml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

(limited to 'run-all-junit-tests')

diff --git a/run-all-junit-tests/pom.xml b/run-all-junit-tests/pom.xml
index 583acfae1..30df18d38 100644
--- a/run-all-junit-tests/pom.xml
+++ b/run-all-junit-tests/pom.xml
@@ -198,7 +198,10 @@
 						<artifactId>maven-surefire-plugin</artifactId>
 						<configuration>
 							<testFailureIgnore>true</testFailureIgnore>
-							<argLine>${jvm.arg.addOpens}</argLine>
+							<argLine>
+								${jvm.arg.addOpens}
+								${jvm.arg.allowSecurityManager}
+							</argLine>
 						</configuration>
 						<executions>
 							<execution>
@@ -253,4 +256,3 @@
 	</build>
 
 </project>
-
-- 
cgit v1.2.3