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

[Vladimir Kozlov]

Looks good to me.

Thanks,
Vladimir

On 4/16/14 4:58 AM, Lev Priima wrote:
> 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
>>
>