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

Seán Coffey sean.coffey at oracle.com
Tue Jan 5 15:36:39 UTC 2016


Looks ok to me also. Please remember to file an approval request[1] 
before pushing to the jdk8u-dev forest. The bug report will need the 
9-na label. It should also be linked to the JDK 9 bug record.

[1] http://openjdk.java.net/projects/jdk8u/approval-template.html

Regards,
Sean.

On 05/01/2016 14:39, Roger Riggs wrote:
> A jdk8u Reviewer is needed for this.
>
> Roger
>
>
>
> On 12/24/2015 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.
>>>>>
>>>>
>>>
>>
>



More information about the jdk8u-dev mailing list