RFR(S) : 8156681 : Add jtreg wrapper for hotspot gtest tests

[Jesper Wilhelmsson]

Thanks!

Looks good!

/Jesper

Den 12/5/16 kl. 17:41, skrev Igor Ignatyev:
> here it is http://cr.openjdk.java.net/~iignatyev/8156681-diif/webrev.00/
>
> — Igor
>
>> On May 12, 2016, at 6:35 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
>>
>> Igor,
>>
>> Do you mind creating a diff between the previous version and this new version?
>>
>> Thanks,
>> /Jesper
>>
>> Den 12/5/16 kl. 17:30, skrev Igor Ignatyev:
>>> Jesper,
>>>
>>> 1st of all, thank you for review.
>>>
>>> unfortunately, testing in our infra reveled some problems w/ the wrapper:
>>> - gtest test library can be located right in ‘test.nativepath’ or in ‘test.nativepath’/../../gtest
>>> - the wrapper pass all flags to gtestLauncher including flags which are not JVM flags, gtestLauncher pass all these args to JNI_CreateJavaVM, so we fail w/ 'Unrecognized option'
>>>
>>> I have updated the webrev : http://cr.openjdk.java.net/~iignatyev/8156681/webrev.01/
>>>
>>> could you please review the updated version?
>>>
>>> Thanks,
>>> — Igor
>>>
>>>> On May 10, 2016, at 8:32 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
>>>>
>>>> Looks good!
>>>> /Jesper
>>>>
>>>> Den 10/5/16 kl. 18:31, skrev Igor Ignatyev:
>>>>> http://cr.openjdk.java.net/~iignatyev/8156681/webrev.00/
>>>>>> 71 lines changed: 71 ins; 0 del; 0 mod;
>>>>>
>>>>> Hi all,
>>>>>
>>>>> could you please integrate this small fix which adds a jtreg wrapper to run gtest tests?
>>>>>
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8156681
>>>>> webrev: http://cr.openjdk.java.net/~iignatyev/8156681/webrev.00/
>>>>>
>>>>> Thanks,
>>>>> — Igor
>>>>>
>>>
>