On Thu, 30 Mar 2023 14:21:29 GMT, Eirik Bjorsnos <d...@openjdk.org> 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` implementation class is moved into `jdk.jartool` module. >> - The `InvalidJarIndexError` exception class is removed because it falls out >> of use >> - The test `test/jdk/sun/misc/JarIndex/metaInfFileNames/Basic.java` is >> removed because it depends on the JarIndex feature being present >> - The test `test/jdk/sun/misc/JarIndex/JarIndexMergeForClassLoaderTest.java` >> is removed because it depends on the JarIndex feature being present >> - The test `test/jdk/sun/misc/JarIndex/JarIndexMergeTest.java` is removed >> because it end up being the only caller of the JarIndex.merge feature >> - All `JarIndex` methods/constructors which are not used by the `jar -i` >> implementation are removed. >> - `JarIndex` is given package-private access. >> >> Outstanding code work: >> >> - Create tests for `JarFile` and `JarInputStream` accepting dusty INDEX jars. >> >> Outstanding work: >> >> - CSR for the removal >> - Release notes for the removal >> - Coordination of the update of the Jar File Specification > > Eirik Bjorsnos has updated the pull request incrementally with one additional > commit since the last revision: > > Include a warning saying that 'The JAR index (META-INF/INDEX.LIST) is > ignored at run-time since JDK 18' Marked as reviewed by lancea (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/13158#pullrequestreview-1371573197
