[9] RFR (XS): 8039260: c.o.j.t.ProcessTools::createJavaProcessBuilder(boolean, String... ) must also take test.java.opts

[Lev Priima]

adding other aliases explicitly

Lev

On 04/11/2014 03:46 PM, Lev Priima wrote:
> Hi all,
>
> Please review small patch:
>
> http://cr.openjdk.java.net/~iignatyev/lpriima/8039260/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8039260
> 6 lines changed: 0 ins; 3 del; 3 mod;
>
> Problem:
> method c.o.j.t.ProcessTools::createJavaProcessBuilder(boolean 
> addTestVmOptions, String... command) starts "java" launcher without 
> passing to it "test.java.opts" received from jtreg harness.
>
> Fix:
> Add both "test.vm.opts" and "test.java.opts" to newly created java 
> process by new parameter name "addTestVmAndJavaOptions"
>
> Testing: jprt default set + all hotspot regression tests
>