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 lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>