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

Reply via email to