[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] [9] Review Request: 8181894 java.desktop module documentation has links to tech
From:       Mandy Chung <mandy.chung () oracle ! com>
Date:       2017-06-17 6:11:27
Message-ID: 8C67D508-9FB7-49BC-8339-CD1DDE0DDBE4 () oracle ! com
[Download RAW message or body]

It's in my build:
   docs/specs/jar/jar.html

Do you pull in the up-to-date changeset? 

Mandy

> On Jun 16, 2017, at 8:11 PM, Sergey Bylokhov <sergey.bylokhov@oracle.com> wrote:
> 
> Hi, Mandy.
> I  rechecked this fix on top of JDK-8150681, and found that the jar/jar.html was \
> not copied to the "/images/docs/specs/ « but for example  «serialization » is \
> copied.  
> 
> > > - The links to a.html were changed to  «{@docRoot}/specs/jar.html » see \
> > > JDK-8150681 
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8181894 \
> > > <https://bugs.openjdk.java.net/browse/JDK-8181894> Webrev can be found at: \
> > > http://cr.openjdk.java.net/~serb/8181894/webrev.00 \
> > > <http://cr.openjdk.java.net/~serb/8181894/webrev.00> 
> > 
> > + * <a href="{@docRoot}/specs/jar.html">JAR File Specification</a>.
> > This should be 
> > 
> > <a href="{@docRoot}/specs/jar/jar.html">
> > 
> > You should do a build to verify.
> > Mandy
> 


[Attachment #3 (unknown)]

<html><head><meta http-equiv="Content-Type" content="text/html \
charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; \
-webkit-line-break: after-white-space;" class="">It's in my build:<div \
class="">&nbsp; &nbsp;docs/specs/jar/jar.html<br class=""><br class="">Do you pull in \
the up-to-date changeset?&nbsp;</div><div class=""><br class=""><div \
class="">Mandy</div><div class=""><br class=""><div><blockquote type="cite" \
class=""><div class="">On Jun 16, 2017, at 8:11 PM, Sergey Bylokhov &lt;<a \
href="mailto:sergey.bylokhov@oracle.com" class="">sergey.bylokhov@oracle.com</a>&gt; \
wrote:</div><br class="Apple-interchange-newline"><div class=""><meta \
http-equiv="Content-Type" content="text/html charset=utf-8" class=""><div \
style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: \
after-white-space;" class=""><div class="">Hi, Mandy.</div><div class="">I \
&nbsp;rechecked this fix on top of JDK-8150681, and found that the jar/jar.html was \
not copied to the "/images/docs/specs/ « but for example  «serialization » is \
copied.&nbsp;</div><div class=""><br class=""></div><div class=""><br \
class=""><blockquote type="cite" class=""><div style="word-wrap: break-word; \
-webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div \
class=""><blockquote type="cite" class=""><div class=""><div style="word-wrap: \
break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" \
class=""><div class=""><div class="">&nbsp;- The links to a.html were changed to  \
«{@docRoot}/specs/jar.html » see&nbsp;JDK-8150681</div><div class=""><br \
class="">Bug:&nbsp;<a href="https://bugs.openjdk.java.net/browse/JDK-8181894" \
class="">https://bugs.openjdk.java.net/browse/JDK-8181894</a><br class="">Webrev can \
be found at:&nbsp;<a href="http://cr.openjdk.java.net/~serb/8181894/webrev.00" \
class="">http://cr.openjdk.java.net/~serb/8181894/webrev.00</a></div></div><div \
class=""><br class=""></div></div></div></blockquote></div><br class=""><div \
class=""><pre class=""><span class="new">+ * &lt;a \
href="{@docRoot}/specs/jar.html"&gt;JAR File \
Specification&lt;/a&gt;.</span></pre><div class="">This should \
be&nbsp;</div></div><div class=""><span class="new">&nbsp;</span></div><div \
class=""><span class="new">&lt;a \
href="</span>{@docRoot}/specs/jar/jar.html"&gt;</div><div class=""><div class=""><br \
class=""></div></div><div class="">You should do a build to verify.</div><div \
class="">Mandy</div></div></blockquote></div><br \
class=""></div></div></blockquote></div><br class=""></div></div></body></html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic