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

[Erik Joelsson]

Hello Mandy,

Looking at this, I just realized that 
$(JDK_OUTPUTDIR)/modules/jdk.dev/com/sun/tools/jdeps/resources/jdeps-modules.xml 
is a generated resource for a module and that you correctly added it to 
the gendata target. Then to make it fit with the new makefile model, the 
running of TOOL_GENMODULESXML should be moved to 
jdk/make/gendata/Gendata-jdk.dev.gmk, which would make it be run 
automatically with correct dependencies. ModulesXml.gmk should also 
probably be renamed to something better describing the checkdeps target, 
which is all it would be doing then. Perhaps it would also fit better in 
the root make dir.

I can understand if fixing the cross compilation issue is urgent and am 
fine with you pushing this to fix that, but would like to see it further 
improved eventually.

/Erik

On 2014-08-27 00:40, Mandy Chung wrote:
> JDK-8055856: checkdeps build target doesn't work for cross-compilation 
> builds
> JDK-8056113: [build] tools.jar missing modules.xml
>
> Webrev at:
>    http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8055856/
>
> This patch fixes a few things about modules.xml
>
> 1. jdeps is invoked at build time to verify the module boundaries.
>    For cross-compilation, it has to run on the host system.
>    jdeps is added in the langtools interim build so that it
>    can run with the BOOT_JDK.  jdeps is also modified to
>    support a system property to specify the path to modules.xml
>    generated later.
>
> 2. The generated modules.xml is solely for jdeps to use until
>    the module system is in place.  The build tool takes the
>    modules.xml in the top repo and generates a new file
>    to include the module membership for jdeps to use.  It
>    is better to name it differently to jdeps-modules.xml
>    to avoid confusion as Magnus suggests.
>
> 3. main-jars is missing the dependency of modules-xml target
>    and thus jdeps in the JDK images fails.  To help catch
>    this issue, jdeps now throws an exception if jdeps-modules.xml
>    is missing.
>
> thanks
> Mandy
>