The package info for com.sun.security.jgss in jdk.security.jgss is also missing. The updated change looks like this (I omit the copyright header):
src/jdk.jartool/share/classes/jdk/security/jarsigner/package-info.java: +/** + * This package defines APIs for signing jar files. + */ +package jdk.security.jarsigner; src/jdk.jartool/share/classes/module-info.java: + * This module also defines APIs for signing JAR files. src/jdk.security.jgss/share/classes/com/sun/security/jgss/package-info.java: +/** + * This package defines classes and interfaces for the JDK extensions + * to the GSS-API. + */ package com.sun.security.jgss; src/jdk.security.jgss/share/classes/module-info.java: /** - * Defines Java extensions to the GSS-API and an implementation of the SASL + * Defines JDK extensions to the GSS-API and an implementation of the SASL * GSSAPI mechanism. Thanks Max > On Aug 31, 2017, at 11:10 PM, Sean Mullan <sean.mul...@oracle.com> wrote: > > On 8/31/17 4:49 AM, Weijun Wang wrote: > >> /** >> * This package contains the {@link jdk.security.jarsigner.JarSigner} API, >> * which backs the signing function of the {@code jarsigner} tool. >> */ > > I think you should say something about that this API can be used to sign JAR > files. The fact that it is also used by jarsigner seems less important to > mention in the package description. I suggest changing this to: > > "This package contains the {@link jdk.security.jarsigner.JarSigner} API which > can be used to sign jar files." > > --Sean