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