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

[Erik Joelsson]

On 2014-08-27 16:39, Mandy Chung wrote:
>
> On 8/27/2014 3:19 AM, Magnus Ihse Bursie wrote:
>> On 2014-08-27 10:26, Erik Joelsson wrote:
>>> 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.
>>
>> I agree. The changes look good, but as Erik suggests, they can be 
>> taken even further, possibly in a separate fix. Perhaps the name 
>> "CheckModules.gmk" would be better suite than ModulesXml.gmk, when 
>> the gendata part has been separated out?
>
> jdeps-modules.xml contains the membership information of all modules.  
> I considered adding Gendata-jdk.dev.gmk but it would have to the last 
> target to run for building exploded image (after jdk.dev classes are 
> compiled).   Do you think that'd be doable?
>
> Currently modules-xml has a dependency on "java" target.
>
It's still doable and would work. You would need the extra explicit 
dependency "jdk.dev-gendata: java", but I can't see any problem with that.

/Erik