Re: CompileJavaModule.gmk overrides values from a custom extension gmk

2017-09-06 Thread Magnus Ihse Bursie
On 2017-08-31 09:43, Erik Joelsson wrote: Updated webrev with the below corrected: http://cr.openjdk.java.net/~erikj/8186983/webrev.02/ Looks good to me. Thanks Jason and Erik! /Magnus /Erik On 2017-08-30 16:57, Erik Joelsson wrote: Hello, On 2017-08-30 16:48, Gary Adams wrote: Is

Re: RFR 8148371: Remove policytool

2017-09-06 Thread Alan Bateman
On 06/09/2017 05:17, Weijun Wang wrote: Hi All Please review the change, which spans to root, jdk and langtools repos. http://cr.openjdk.java.net/~weijun/8148371/ I've searched for the "policytool" word in the whole jdk10/jdk10 forests, removed all files having the word inside the path

Re: RFR 8148371: Remove policytool

2017-09-06 Thread Erik Joelsson
From a build point of view this looks good. /Erik On 2017-09-06 06:17, Weijun Wang wrote: Hi All Please review the change, which spans to root, jdk and langtools repos. http://cr.openjdk.java.net/~weijun/8148371/ I've searched for the "policytool" word in the whole jdk10/jdk10 forests,