Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-10 Thread Eirik Bjorsnos
On Mon, 10 Apr 2023 10:09:04 GMT, Lance Andersen wrote: > I think we are good to go. Thanks, the PR is now ready to be sponsored by a Committer. - PR Comment: https://git.openjdk.org/jdk/pull/13158#issuecomment-1501666039

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-10 Thread Lance Andersen
On Fri, 7 Apr 2023 07:47:49 GMT, Eirik Bjorsnos wrote: >> This PR removes the JAR index feature from the runtime: >> >> - `URLClassPath` is updated to remove the `enableJarIndex` system property >> and any code which would be called when this property was `true` >> - The `JarIndex` implementat

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-10 Thread Lance Andersen
On Fri, 7 Apr 2023 07:47:49 GMT, Eirik Bjorsnos wrote: >> This PR removes the JAR index feature from the runtime: >> >> - `URLClassPath` is updated to remove the `enableJarIndex` system property >> and any code which would be called when this property was `true` >> - The `JarIndex` implementat

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-09 Thread Eirik Bjorsnos
On Fri, 7 Apr 2023 07:47:49 GMT, Eirik Bjorsnos wrote: >> This PR removes the JAR index feature from the runtime: >> >> - `URLClassPath` is updated to remove the `enableJarIndex` system property >> and any code which would be called when this property was `true` >> - The `JarIndex` implementat

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-07 Thread Eirik Bjorsnos
On Fri, 7 Apr 2023 16:29:31 GMT, Alan Bateman wrote: > I've updated the CSR to align it with the proposed solution. Please review > again and if you happy, press "Finalize". Thanks, Alan, this looks good to me. I have pressed "Finalize". The CSR makes promises about the Release Note. Do you th

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-07 Thread Alan Bateman
On Fri, 7 Apr 2023 07:47:49 GMT, Eirik Bjorsnos wrote: >> This PR removes the JAR index feature from the runtime: >> >> - `URLClassPath` is updated to remove the `enableJarIndex` system property >> and any code which would be called when this property was `true` >> - The `JarIndex` implementat

Re: RFR: 8302819: Remove JAR Index [v6]

2023-04-07 Thread Eirik Bjorsnos
> This PR removes the JAR index feature from the runtime: > > - `URLClassPath` is updated to remove the `enableJarIndex` system property > and any code which would be called when this property was `true` > - The `JarIndex` implementation class is moved into `jdk.jartool` module. > - The `Invalid