RFR: 8076417: Update test/jdk/asm/AsmSanity.java with modules

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Wed Apr 19 18:26:49 UTC 2017


Made the change you recommended, and here is the updated webrev
http://cr.openjdk.java.net/~ksrini/8076417/webrev.01/

Thanks
Kumar

> Hi, Kumar
>
> As the removed test is the only one test under jdk/asm, it needs to be 
> removed from jdk_other test group:
>
> --- a/test/TEST.groups
> +++ b/test/TEST.groups
> @@ -269,7 +269,6 @@
>      javax/transaction \
>      javax/xml \
>      -javax/xml/crypto \
> -    jdk/asm \
>
>
> Thanks,
> Amy
>
> On 4/19/17 2:39 AM, Kumar Srinivasan wrote:
>> Hello,
>>
>> Please review [1] which addresses bug [2]. This test is unnecessary, 
>> as there
>> are other components and tests within the jdk, which will fail if ASM
>> is not included, therefore this test is  removed.
>>
>> Thanks
>> Kumar
>>
>> [1] http://cr.openjdk.java.net/~ksrini/8076417/webrev.00/
>> [2] https://bugs.openjdk.java.net/browse/JDK-8076417
>



More information about the core-libs-dev mailing list