Review request: 8055856: checkdeps build target doesn't work for cross-compilation builds

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Thu Aug 28 08:32:19 UTC 2014


On 2014-08-27 18:00, Mandy Chung wrote:
> Erik, Magnus,
>
> This is much easier than I have thought.  I really like this new build.
Glad to hear! :)
> I have separated out Gendata-jdk.dev.gmk and removed the modules-xml
> target completely.
>
> Webrev at:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8055856/webrev.01/

Looks good, but I still think the CheckModules.gmk file should be in 
$TOP/make/ instead of jdk/make. As a rule of thumb, stuff working on 
"all modules" should sit in the top level, but stuff working on a 
specific module should sit close to that module. Apart from that, it 
looks great.

/Magnus



More information about the core-libs-dev mailing list