On 8/28/14 3:28 AM, Wang Weijun wrote:
This is the sub-task of "8042900: Allow com.sun.security.jgss to be in different module than org.ietf.jgss" that actually moves the files.Code changes include 2 parts: 1. For jdk9/dev repo: http://cr.openjdk.java.net/~weijun/8056141/webrev.00/ 2. For jdk9/dev/jdk. It's just moving everything inside src/java.security.jgss/share/classes/com/sun/security/ to src/jdk.security.jgss/share/classes/com/sun/security. This includes all classes in the com/sun/security/jgss and com/sun/security/sasl/gsskerb packages.
Your change looks okay including the change to modules.xml. I believe jdk.security.jgss using sun.security.krb5.internal is introduced by 8042900. Can you run the following and double check: $ cd BUILD_OUTPUTDIR/jdk $ ./bin/jdeps -M modules/java.security.jgss modules/jdk.security.jgss The output will tell you what modules java.security.jgss and jdk.security.jgss depend on and then follow with the package-level dependences. Looks for "JDK internal API" that indicates that you need to add a qualified export to jdk.security.jgss to use. You have already done this: $ make verify-modules The images build will run the verify-modules as well. Mandy
