RFR(M): 8078896: Add @modules as needed to the jdk_svc tests (original) (raw)
Mandy Chung mandy.chung at oracle.com
Tue May 5 21:00:35 UTC 2015
- Previous message: RFR(M): 8078896: Add @modules as needed to the jdk_svc tests
- Next message: RFR(M): 8078896: Add @modules as needed to the jdk_svc tests
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 05/05/2015 01:33 PM, Mandy Chung wrote:
On 05/05/2015 02:04 AM, Yekaterina Kantserova wrote: Hi,
Could I please have a review of this fix. bug: https://bugs.openjdk.java.net/browse/JDK-8078896 webrev: http://cr.openjdk.java.net/~ykantser/8078896 com.sun.management has been moved to jdk.management module. The patch for JDK-8042901 is just integrated in jdk9/dev today. Most, if not all, test/com/sun/management tests need updates to require @modules jdk.management. There are several test/java/lang/management tests dependng on both java.lang.management and com.sun.management. For tests only using java.lang.management API, @modules java.management is adequate. test/com/sun/jdi/InterfaceMethodsTest.java test/com/sun/jdi/InterruptHangTest.java Since you have updated the end year of the copyright header in most tests, I spot a couple and so mention it to you. Otherwise looks good. Thanks for doing this.
About the test selection, one typical aspect of svc tests is to run a j* tool in a child process (e.g. jinfo, jstack, jstat, jstatd,jcmd, jps etc that are in jdk.jcmd module). I would expect all test/sun/tools/jcmd tests should have @modules jdk.jcmd and similiar for other sun/tools/j* tests. Are you thinking to come back in the next round of test selection update?
Mandy
- Previous message: RFR(M): 8078896: Add @modules as needed to the jdk_svc tests
- Next message: RFR(M): 8078896: Add @modules as needed to the jdk_svc tests
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]