Re: RFR JDK-8213909: jdeps --print-module-deps should report missing dependences

2018-11-21 Thread Sundararajan Athijegannathan
Hi Mandy, Delta webrev looks good. -Sundar On 21/11/18, 2:12 AM, Mandy Chung wrote: Hi Sundar, Per Joe's feedback on this CSR, a new `--no-recursive` option is added to restore the non-transitive behavior. Here is the delta webrev:

Re: RFR JDK-8213909: jdeps --print-module-deps should report missing dependences

2018-11-20 Thread Mandy Chung
Hi Sundar, Per Joe's feedback on this CSR, a new `--no-recursive` option is added to restore the non-transitive behavior.  Here is the delta webrev: http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8213909/webrev.01-delta Thanks Mandy On 11/19/18 9:22 AM, Mandy Chung wrote: Thanks Sundar. 

Re: RFR JDK-8213909: jdeps --print-module-deps should report missing dependences

2018-11-19 Thread Mandy Chung
Thanks Sundar.  Here is the CSR:    https://bugs.openjdk.java.net/browse/JDK-8213915 Mandy On 11/19/18 6:48 AM, Sundararajan Athijegannathan wrote: Looks good to me. -Sundar On 15/11/18, 5:46 AM, Mandy Chung wrote: This patch improves `jdeps --print-module-deps`, `--list-deps` and

Re: RFR JDK-8213909: jdeps --print-module-deps should report missing dependences

2018-11-19 Thread Sundararajan Athijegannathan
Looks good to me. -Sundar On 15/11/18, 5:46 AM, Mandy Chung wrote: This patch improves `jdeps --print-module-deps`, `--list-deps` and `--list-reduced-deps` to report missing dependences and also do transitive dependence analysis as the default. Webrev at: