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

[Lev Priima]

Thanks, Vladimir.

Igor, Could you please push it?

Lev

On 04/16/2014 10:01 PM, Vladimir Kozlov wrote:
> 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
>>>
>>