Thanks for the update. We have some workarounds in the Groovy codebase too. I'll try to tidy them up too once this has settled.
Thanks again, Paul. On Mon, May 15, 2023 at 10:08 PM Jaikiran Pai <jaiki...@apache.org> wrote: > Hello Paul, > > On 12/12/22 5:30 am, Paul King wrote: > > Do you know if there is an issue with the "allow" class approach if > > multiple projects adopt that technique? E.g. if Netbeans or Groovy > > also have an allow class, will that cause a split package violation or > > since it isn't really referenced except for those early JDKs, that we > > should be okay? I will eventually try this out myself if searching > > doesn't help, but just wondering if someone has already checked this. > > The use of a "allow" class as a workaround to older versions of JDK > considering this value as a classname for -Djava.security.manager system > property, was always a brittle one. As such, Oracle JDK in its upcoming > October CPU release is going to introduce a change which will treat the > values "allow" and "disallow" specially (by ignoring them and not > considering them as a classname) for the java.security.manager system > property. This will be available in Oracle's 11, 8 and 7 releases and is > being tracked in https://bugs.openjdk.org/browse/JDK-8301118. Hopefully > other vendors too will bring in this change in their releases. > > What that will then mean is that, applications/users will no longer have > to first detect the version of Java before deciding whether or not they > can set the value "allow" for the java.security.manager system property > (if at all they want to set that value). > > As a related note, after Ant 1.10.13 was released we have received > reports that the "allow" workaround we introduced, has its own set of > issues. It was always a temporary change in Ant to allow for this > version of Ant to work against recent releases of Java. I'm in the > process of undoing this "allow" workaround and then completing skipping > setting of SecurityManager against recent versions of Java, in Ant. > > -Jaikiran > > >
