RFR: 8273401: Remove JarIndex support in URLClassPath [v2]
github.com+53162078+shiyuexw at openjdk.java.net
Wed Sep 15 01:48:46 UTC 2021
On Wed, 8 Sep 2021 06:30:34 GMT, wxiang <github.com+53162078+shiyuexw at openjdk.org> wrote:
>> wxiang has updated the pull request incrementally with one additional commit since the last revision:
>> Some minor changes
>> 1. remove public for INDEX_NAME in JarFile
>> 2. recover the definition for INDEX_NAME in JarIndex
>> 3. mv the test case JarIndexMergeTest.java into test/jdk/java/util/jar
> Create CSR: https://bugs.openjdk.java.net/browse/JDK-8273473
> @shiyuexw I discussed this issue with other maintainers in this area. There was agreement on dropping the support from the URLClassLoader implementation but it was suggested that it should be disabled for a release or two before the code is removed. A system property can be used to re-enable. I've updated the CSR to reflect this proposal. Are you okay to continue with this new proposal? It would mean that we wouldn't remove the tests but instead change them to run with the system property.
Yes, I will take care of it. Need I create a new PR?
More information about the core-libs-dev