RFR(S) : 8051896 : jtreg tests don't use $TESTJAVAOPTS

[Vladimir Kozlov]

The order of flags on command line is important. I think TESTJAVAOPTS 
should be last to take priority:

TESTOPTS="${TESTVMOPTS} ${TESTJAVAOPTS}"

Please, add copyright headers to files in compiler/6894807.

Otherwise look good.

Thanks,
Vladimir

On 7/24/14 1:37 PM, Igor Ignatyev wrote:
> // was Re: RFR(XS) : 8051896 : compiler/ciReplay tests don't use
> $TESTJAVAOPTS:
>
> updated webrev: http://cr.openjdk.java.net/~iignatyev/8051896/webrev.01/
> 93 lines changed: 15 ins; 40 del; 38 mod;
>
> On 07/24/2014 10:18 PM, Vladimir Kozlov wrote:
>> Looks good.
>>
>> Is not this a general problem for all our tests? They use only
>> TESTVMOPTS.
> Yeap, it's a general problem, I've updated all tests.
> I thought jtreg merges TESTVMOPTS and TESTJAVAOPTS flags
>> together.
>>
>> Thanks,
>> Vladimir
>>
>> On 7/24/14 11:04 AM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8051896/webrev.00/
>>> 12 lines changed: 2 ins; 0 del; 10 mod
>>>
>>> Hi all,
>>>
>>> Please review patch:
>>>
>>> Problem:
>>> the tests use only TESTVMOPTS, but jtreg propagates some flags by
>>> TESTJAVAOPTS variable
>>>
>>> Fix:
>>> usages of TESTVMOPTS were replaced by TESTOPTS which is initialized as
>>> concatenated values of TESTVMOPTS and TESTJAVAOPTS
>>>
>>> jbs: https://bugs.openjdk.java.net/browse/JDK-8051896
>>> testing: jprt