RFR(S): 8144937 [TEST_BUG] testlibrary_tests should be excluded for compact1 and compact2 execution

[Roger Riggs]

+1

Thanks, Roger


On 12/24/15 5:05 AM, Sergei Kovalev wrote:
> Hi Roger,
>
> Thank you for notice this.
> I've created new patch based on jdk8u-dev.
>
> http://cr.openjdk.java.net/~skovalev/8144937/webrev.01/
>
> The fix is similar in approach but has little difference in 
> implementation due to differences in the content of TEST.groups file 
> for JDK9 and JDK8u.
>
> 23.12.15 21:32, Roger Riggs wrote:
>> Hi Sergei,
>>
>> The diff seems to be against a jdk9 repo in contrast to the comment 
>> that says it is targeted at 8u.
>>
>> But otherwise looks ok.
>>
>> Roger
>>
>> On 12/23/2015 4:42 AM, Sergei Kovalev wrote:
>>> Could somebody take a look?
>>>
>>> 21.12.15 15:44, Sergei Kovalev wrote:
>>>> Hello All,
>>>>
>>>> Please review a fix for 
>>>> https://bugs.openjdk.java.net/browse/JDK-8144937
>>>> Webrev: http://cr.openjdk.java.net/~skovalev/8144937/webrev.00/
>>>>
>>>> Problem: several fixes should be excluded from execution on compact 
>>>> profiles. Also some library classes required "sun.tools.jar.Main" 
>>>> class that not exists in JRE.
>>>>
>>>> The CR targeted for 8u76. For JDK9 required separate fix based on 
>>>> modules that not applicable for JDK8u.
>>>> Fix verified by jtreg test list generation.
>>>>
>>>
>>
>