RFR: JDK-8189611: JarFile versioned stream and real name support
Alan Bateman
Alan.Bateman at oracle.com
Tue Nov 21 11:25:06 UTC 2017
On 21/11/2017 02:58, Xueming Shen wrote:
> :
>
> http://cr.openjdk.java.net/~sherman/8189611/webrev
>
> jdeps' VersionHelper.java still accesses the "getRealName()" via the
> SharedSecrets.
> Since jdeps is being compiled/built with the bootjdk, I'm leaving it
> untouched for
> now.
>
I think we need to create an issue in JIRA to get jdeps updated. It
actually has two issues, the other is the usage of
jdk.internal.util.jar.VersionedStream. I'm surprised that there are
issues with the boot JDK because it uses both in JDK 9 when the boot JDK
is JDK 8. No issue with keeping this separate to the current effort.
I've skimmed through the latest webrev and it looks good good. The
JarEntry::getRealName and JarFile::versionedStream APIs look good.
The update to jlink to use the standard API should mean that java.base
no longer needs to export jdk.internal.util.jar to jdk.jlink.
JarFile is updated to import LinkedHashMap, I don't think that is needed
in this version.
I think that is all I have for now.
-Alan
More information about the core-libs-dev
mailing list