RFR: JDK-8181298: Assertion failure in com.sun.tools.javac.comp.Modules

[Erik Joelsson]

Looks good to me as well.

/Erik


On 2017-07-12 16:58, Jan Lahoda wrote:
> Hi,
>
> The ct.sym building needs to read existing module-infos, to properly 
> detect transitive dependencies. But the target does not depend on all 
> module-infos being generated, which may in turn lead to problems with 
> incomplete module graph.
>
> So, the proposed patch is to add the dependency on all generated 
> module-infos. javac should crash this way even in a presence of 
> incomplete module graph, this will be fixed under a separate ticket.
>
> The cause was found and patch is by Erik.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8181298
> Webrev: http://cr.openjdk.java.net/~jlahoda/8181298/webrev.00/
>
> Comments are welcome.
>
> Thanks,
>     Jan