Re: [9] Review Request: 8072663: Remove the sun.security.acl package which is not used in the JDK

2015-02-17 Thread Alan Bateman
On 17/02/2015 20:59, Jason Uh wrote: Please review this fix, which removes the sun.security.acl package. webrev: http://cr.openjdk.java.net/~juh/8072663/00/ jbs: http://bugs.openjdk.java.net/browse/JDK-8072663 The sun.security.acl package is the default implementation of java.security.acl but

Re: Apache Maven JDeps Plugin

2015-02-17 Thread Mandy Chung
Hi Robert, Indeed this looks very useful. On 2/16/2015 10:45 AM, Alan Bateman wrote: On 16/02/2015 18:28, Robert Scholte wrote: Hi Alan, if you are referring to the -R / -recursive option of the jdeps tool, then yes you can. See http://maven.apache.org/plugins-archives/maven-jdeps-plugin-LA

Re: [9] Review Request: 8072663: Remove the sun.security.acl package which is not used in the JDK

2015-02-17 Thread Sean Mullan
Looks fine to me (pending CCC approval). --Sean On 02/17/2015 03:59 PM, Jason Uh wrote: Please review this fix, which removes the sun.security.acl package. webrev: http://cr.openjdk.java.net/~juh/8072663/00/ jbs: http://bugs.openjdk.java.net/browse/JDK-8072663 The sun.security.acl package is

[9] Review Request: 8072663: Remove the sun.security.acl package which is not used in the JDK

2015-02-17 Thread Jason Uh
Please review this fix, which removes the sun.security.acl package. webrev: http://cr.openjdk.java.net/~juh/8072663/00/ jbs: http://bugs.openjdk.java.net/browse/JDK-8072663 The sun.security.acl package is the default implementation of java.security.acl but it's not used in JDK. The JCK tests fo

Re: Apache Maven JDeps Plugin

2015-02-17 Thread Robert Scholte
Hi Tibor, the plugin is just a thin wrapper around the jdeps executable to make it easier to integrate it with your Maven project. So for the Apache Maven JDeps plugin it is out of scope. However, you could share your idea with the jigsaw project[1] thanks, Robert [1] http://mail.openjdk

Re: Apache Maven JDeps Plugin

2015-02-17 Thread Alan Bateman
On 16/02/2015 23:04, Robert Scholte wrote: Hi Alan, I've added a flag called failOnWarning (default:true), assuming that the usage of jdkinternals is considered a warning and not an error. With the following configuration you'll be able to run jdeps multiple times within the same build. This