> On Mar 30, 2016, at 11:05 PM, Alan Bateman <alan.bate...@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
