RFR: JDK-8058118: Generate modules.list during the build

[Chris Hegarty]

On 12 Sep 2014, at 20:04, Mandy Chung <mandy.chung at oracle.com> wrote:

> On 9/12/14 7:03 AM, Erik Joelsson wrote:
>> Hello,
>> 
>> The checked in modules.list file defines the dependencies between modules for the build. The dependency information in this file is already captured in the modules.xml. Rather than keeping two copies of this information, with this change, modules.list is generated during the build.
>> 
>> Chris provided the java implementation for the actual list generation. I've made the makefile changes.
>> 
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8058118
>> Webrevs: http://cr.openjdk.java.net/~erikj/8058118/
> 
> Looks okay. A couple minor comments on the build tool:

Update jdk part as per Mandy’s comments:

  http://cr.openjdk.java.net/~chegar/8058118/webrev_jdk.01/webrev/

-Chris.

> build/tools/module/GenModulesList.java
>   line 104: maybe s%x%d% to make it little clear as dependence.
> 
> filter(GenModulesList::isNotAggregator) is done in several places.
> Would it be better to do the filtering when writing out modules.list
> in line 85 and 86?
> 
> Nit: line 82 - it may be clearer to rename sort to topoSort to make
> it clear or simply inline line 98-99.
> 
> Mandy
>