<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Little remainder.<br>
<br>
<div class="moz-cite-prefix">On 06.05.2016 21:14, Svetlana
Nikandrova wrote:<br>
</div>
<blockquote cite="mid:572CDEEE.80108@oracle.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
Hello,<br>
<br>
I would be grateful if someone could find time to review this
backport of tests enhancement.<br>
I received few off-line notes and decided to rework this backport
a little.<br>
<br>
Here is my new webrev:<br>
<a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.01/">http://cr.openjdk.java.net/~snikandrova/8049171/webrev.01/</a><br>
<br>
List of changes from original patch:<br>
1) No ProcessTools.java changes in webrev as they have been
backported earlier. <br>
2) Utils.java : new utility method <span class="new">transferBetweenStreams</span>
meant to be used instead of InputStream's transferTo method that
is present since 9. <br>
3)
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
JarUtils.java: changes in createJar method to consider a situation
then files aren't located in current directory.<br>
4) NestedActions.java: refactored to use JarUtils method to create
jars.<br>
5) TimestampCheck.java - omitted
System.getProperty("test.tool.vm.opts") that is not used in jdk 8
testing. <br>
6) Used System.getProperty("test.jdk") instead of
System.getProperty("java.home") in tests.<br>
<br>
Thank you, <br>
Svetlana <br>
<br>
<div class="moz-cite-prefix">On 28.04.2016 20:35, Svetlana
Nikandrova wrote:<br>
</div>
<blockquote cite="mid:572249EB.8090808@oracle.com" type="cite">Hello,
<br>
<br>
please review and approve the backport of tests enhancement: <br>
Original tests from <br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8049171">https://bugs.openjdk.java.net/browse/JDK-8049171</a>
<br>
jdk 9 review:
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-January/031006.html">http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-January/031006.html</a><br>
jdk 9 changeset: <a moz-do-not-send="true"
class="moz-txt-link-freetext"
href="http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5c61ccd9c162">http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5c61ccd9c162</a>
<br>
Originally reviewed by Max Wang <br>
<br>
Webrev: <br>
<a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/">http://cr.openjdk.java.net/~snikandrova/8049171/webrev.00/</a>
<a moz-do-not-send="true" class="moz-txt-link-rfc2396E"
href="http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/"><http://cr.openjdk.java.net/%7Esnikandrova/8049171/webrev.00/></a>
<br>
<br>
List of changes between original patch and current webrev: <br>
<br>
1) No ProcessTools.java changes as they have been backported
earlier. <br>
2) JarUtils.java : private method transferToOutputStream instead
ofInputStream's transferTo method (present since 9). <br>
3) TimestampCheck.java - changes applied by hand to omit
System.getProperty("test.tool.vm.opts") that is not used in jdk
8 testing <br>
4) Used System.getProperty("test.jdk") instead of
System.getProperty("java.home") <br>
<br>
Tested with JPRT. <br>
<br>
Thank you, <br>
Svetlana <br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>