RFR 8042901: Allow com.sun.management to be in a different module to java.lang.management

Mandy Chung mandy.chung at oracle.com
Thu Apr 2 20:09:08 UTC 2015


On 4/2/15 12:58 PM, Shanliang Jiang wrote:
> Hi,
>
> I have to ask the review again because I need to modify:
> langtools/src/jdk.dev/share/classes/com/sun/tools/jdeps/Profile.java
>
> The issue was found when langtools tests were added into my test list.
>
> The new version is:
>    http://cr.openjdk.java.net/~sjiang/JDK-8042901/02/
>

Thanks for catching the jdeps change.  Profile.java change looks okay 
and this list is temporary.  jdeps will be updated to determine the 
exported APIs at runtime when the module system is moving along.

> which integrated also the Mandy's comments in the following mail.
>
CheckSomeMXBeanImplPackage.java
   line 45-50 & 58-60: should be called unconditionally since they
   should pass if java.management is present.

Okay to push once the test is updated.

Mandy




More information about the build-dev mailing list