RFR 8212807: tools/jar/multiRelease/Basic.java times out

[Brent Christian]

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.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
> 
> 
>