Re: Review request 8069551: Move java.security.acl from compact3 to java.base
On 1/30/15 12:50 AM, Alan Bateman wrote: On 29/01/2015 20:58, Mandy Chung wrote: Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base module rather than complicating the module graph with a close-to-useless dedicated module. This looks okay. One small suggestion is to move the lines in unshuffle_list.txt up to where the other java.base paths are. Yes that's better. Fixed. thanks. Mandy
Re: Review request 8069551: Move java.security.acl from compact3 to java.base
On 29/01/2015 20:58, Mandy Chung wrote: Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base module rather than complicating the module graph with a close-to-useless dedicated module. This looks okay. One small suggestion is to move the lines in unshuffle_list.txt up to where the other java.base paths are. -Alan
Re: Review request 8069551: Move java.security.acl from compact3 to java.base
Looks fine to me. --Sean On 01/29/2015 03:58 PM, Mandy Chung wrote: Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base module rather than complicating the module graph with a close-to-useless dedicated module. Mandy
Review request 8069551: Move java.security.acl from compact3 to java.base
Webrev at: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8069551/webrev.00 java.security.acl is superceded by java.security package since 1.2. This patch proposes to move java.security.acl package to java.base module rather than complicating the module graph with a close-to-useless dedicated module. Mandy