Review request 8153035: GenModuleInfoSource strips away the API

Mandy Chung mandy.chung at oracle.com
Thu Mar 31 06:09:07 UTC 2016


> On Mar 30, 2016, at 11:05 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> On 30/03/2016 20:09, Mandy Chung wrote:
>> GenModuleInfoSource is a build tool that augments module-info.java with platform-specific declaration such as exports, uses and provides and generates the resulting module-info.java in gensrc directory.  This utility should really be a simple one that inserts/appends the extra info to the original module-info.java.  The patch replaces the old code and removes the unused files that were used for modules.xml.
>> 
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8153035/webrev.00/
>> 
> At L83 then the IAE message is "invalid -provide argument", I assume this should be "-provides".
> 
> You want to check the indentation at L71-77 and L81-87 as it looks like it has been pasted in without re-aligning.

Already pushed. I will file a new issue to fix this.

Thanks
Mandy




More information about the core-libs-dev mailing list