[Urgent JDK 9] RFR: 8182844: Fix broken links in the generated jvmti.html

[Langer, Christoph]

Hi Serguei,

the change looks fine and fixes the URLs.

Best regards
Christoph

From: serviceability-dev [mailto:serviceability-dev-bounces at openjdk.java.net] On Behalf Of serguei.spitsyn at oracle.com
Sent: Samstag, 24. Juni 2017 06:06
To: serviceability-dev at openjdk.java.net; Mandy Chung <Mandy.Chung at oracle.com>; Alan Bateman <Alan.Bateman at oracle.com>
Subject: [Urgent JDK 9] RFR: 8182844: Fix broken links in the generated jvmti.html

Please, review this urgent JDK 9 review request for the bug:
  https://bugs.openjdk.java.net/browse/JDK-8182844

Webrev:
  http://cr.openjdk.java.net/~sspitsyn/webrevs/2017/hotspot/8182844-jvmti-spec-links/


Summary:

  The broken links are:
        <a href="http://docs.oracle.com/javase/9/docs/technotes/guides/jpda/architecture.html">Java
        <a href="http://docs.oracle.com/javase/9/jni/invocation.html#getenv">
        <a href="http://docs.oracle.com/javase/9/jni/types.html#modified-utf-8-strings">
        <a href="http://docs.oracle.com/javase/9/jni/design.html">Java
        <a href="http://docs.oracle.com/javase/9/jni/functions.html#local-references">
          <a href="http://docs.oracle.com/javase/9/jni/design.html#java-exceptions">Java Exceptions</a>
      <a href="http://docs.oracle.com/javase/9/jni/invocation.html#attaching-to-the-vm">Attaching to the VM</a>.
            <a href="http://docs.oracle.com/javase/9/jni/types.html#type-signatures">JNI
            <a href="http://docs.oracle.com/javase/9/jni/index.html">JNI Specification</a>
            <a href="http://docs.oracle.com/javase/9/jni/index.html">JNI Specification</a>
          See <a href="http://docs.oracle.com/javase/9/jni/functions.html">JNI
      path to a <a href="http://docs.oracle.com/javase/9/docs/technotes/guides/jar/jar.html">
     <a href="http://docs.oracle.com/javase/9/docs/technotes/guides/jar/jar.html">JAR file</a> to be
     <a href="http://docs.oracle.com/javase/9/jni/functions.html#interface-function-table">

  There are two issues:
    - 3 links have the incorrect part "docs/technotes/guides" that now has to be "docs/specs".
    - the jvmti.xsl injects the obsolete prefix to every link:
        "http://docs.oracle.com/javase/9/"<http://docs.oracle.com/javase/9/>

  The fix is to replace the "docs/technotes/guides" with "docs/specs"
  and skip injecting the incorrect prefix "http://docs.oracle.com/javase/9/"<http://docs.oracle.com/javase/9/>.
  So that all links become relative to the location of the jvmti.html.


  Also, it can be too late to push this at this time.
  Sorry that I've discovered this so late.
  One question is if the priority of this bug has to be raised to P1,
  or maybe this bug needs to be deferred to 9u.


Thanks,
Serguei
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20170624/2e2b7950/attachment-0001.html>