Hi, Lance
This change is to collect more information in case this happens again, yes?
Looks pretty good - just a couple comments:
test/jdk/tools/jar/multiRelease/Basic.java
536 jar("ufm", jarfile, manifest.toString(),
Is there a reason not to convert this to call jarTool() ?
--
test/jdk/tools/jar/multiRelease/MRTestBase.java
L146-L152
indentation looks off-by-one
Thanks,
-Brent
On 5/30/19 9:21 AM, Lance Andersen wrote:
Hi all,
The following fix addresses an issue with an occasional timeout for
tools/jar/multiRelease/Basic.java.
The webrev can be found at:
http://cr.openjdk.java.net/~lancea/8212807/webrev.00/index.html
<http://cr.openjdk.java.net/~lancea/8212807/webrev.00/index.html>
Best,
Lance
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen|
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[email protected] <mailto:[email protected]>
