Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2025-02-05 Thread via GitHub
HannesWell commented on PR #216: URL: https://github.com/apache/ant/pull/216#issuecomment-2637707338 Thank you for your reviews and accepting this. -- 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

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2025-02-04 Thread via GitHub
jaikiran merged PR #216: URL: https://github.com/apache/ant/pull/216 -- 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

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2025-02-04 Thread via GitHub
HannesWell commented on PR #216: URL: https://github.com/apache/ant/pull/216#issuecomment-2635435720 > The reason I say we should avoid the check for Java 12 or such is because although the "allow" and "disallow" values for the "java.security.manager" system property were introduced in that

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2024-12-18 Thread via GitHub
jaikiran commented on PR #216: URL: https://github.com/apache/ant/pull/216#issuecomment-2551818231 Hello Hannes, caching the value is fine and what you propose here is reasonable. My only suggestion would be to remove the reference and usage of Java 12 check. I think if Java version is less

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2024-12-17 Thread via GitHub
HannesWell commented on PR #216: URL: https://github.com/apache/ant/pull/216#issuecomment-2549864805 Thanks for the quick feedback. > 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

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2024-12-15 Thread via GitHub
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 `isSetSecurityManagerAll

Re: [PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2024-12-15 Thread via GitHub
HannesWell commented on PR #216: URL: https://github.com/apache/ant/pull/216#issuecomment-2543892077 I have now changed this to be less intrusive and not relying on deprecated (`SecurityManager`)-API. On the other hand, the previous approach would also handle the case if a security-m

[PR] Consider VM-configuration when determining if SecurityManager may be set [ant]

2024-12-15 Thread via GitHub
HannesWell opened a new pull request, #216: URL: https://github.com/apache/ant/pull/216 Since Java-12 users can configure a JVM to disallow the installation of a SecurityManager at runtime. The default value of the configuration has changed over the following versions and since Java-24 it's