jaikiran commented on PR #216:
URL: https://github.com/apache/ant/pull/216#issuecomment-2543896795
Hello Hannes, I haven't had a chance to look at this in detail, but looking
at the linked issue, I think what should be done here is, only change the
implementation of `isSetSecurityManagerAllowed()` method to:
```java
public static boolean isSetSecurityManagerAllowed() {
// for Java 18 or higher Ant will not set the SecurityManager
if (isJava18OrHigher) {
return false;
}
// for other versions (lower than Java 18), if "java.security.manager"
// system property is set to "disallow", Ant will not set the
SecurityManager
return !"disallow".equals(System.getProperty("java.security.manager"));
}
```
That way Ant will continue to not set the SecurityManager for Java 18 or
higher and for lesser versions, Ant will then not set the SecurityManager, if
the system property is set to "disallow".
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org
