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