Hi Mandy, I've built the docs with your patch and the link is still not resolved. It points to the docs/api/specs/jpda/jpda.html
But it is resolved if the patch is corrected with: + * <a href="${docRoot}/../../specs/jpda/jpda.html"> Is it 9 or 10? Thanks, Serguei On 6/19/17 12:47, Mandy Chung wrote:
Serguei, Can you review this patch fixing the link to JPDA? Mandy diff --git a/src/jdk.jdi/share/classes/module-info.java b/src/jdk.jdi/share/classes/module-info.java --- a/src/jdk.jdi/share/classes/module-info.java +++ b/src/jdk.jdi/share/classes/module-info.java @@ -39,12 +39,13 @@ * creation, etc. The ability to inspect a suspended thread's state, local * variables, stack backtrace, etc. * <p> - * JDI is the highest-layer of the Java Platform Debugger Architecture (JPDA). - * For more information on the Java Platform Debugger Architecture, see the <a - * href="{@docRoot}/../../../../technotes/guides/jpda/index.html"> Java - * Platform Debugger Architecture documentation</a> for this release and the <a - * href="http://java.sun.com/products/jpda">Java Platform Debugger Architecture - * website</a>. + * JDI is the highest-layer of the + * <a href="${docRoot}/../specs/jpda/jpda.html"> + * Java Platform Debugger Architecture (JPDA)</a>. + * <p> + * This module includes a simple command-line debugger, + * <em>{@index jdb jdb tool}</em>. + * * <h3>Global Exceptions</h3> * <p> * This section documents exceptions which apply to the entire API and are thus @@ -102,10 +103,6 @@ * unloaded. * </blockquote> * - * <h3>jdb</h3> - * - * <em>{@index jdb jdb tool}</em> is a simple command-line debugger provided - * in this module. * * <dl style="font-family:'DejaVu Sans', Arial, Helvetica, sans serif"> * <dt class="simpleTagLabel">Tool Guides: @@ -119,6 +116,8 @@ * * @moduleGraph * @since 9 + * @see <a href="${docRoot}/../specs/jpda/jpda.html"> + * Java Platform Debugger Architecture (JPDA)</a> */ module jdk.jdi { requires jdk.attach;