Review Request JDK-8160286: jmod hash is creating unlinkable modules.

[Alan Bateman]

On 11/01/2017 23:47, Mandy Chung wrote:

> Webrev:
>    http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8160286/webrev.00/
>
> jmod and jar -—hash-modules option to specify a pattern of modules
> to be hashed in the module M being created.  It records the modules
> that depend on M directly and indirectly.
>
This looks quite good.  At some point then we'll need to move the tool 
support out of jdk.internal.module but is something for another day.

For ModuleHashesBuilder then it might be useful to put a comment on the 
constructors as it's not immediately obvious why both are needed. Also I 
wonder if we should use a term other than "base" for the modules that 
don't have references to other modules in the sub-graph (they are sort 
of leaf modules in the sub-graph).  A typo at L96 "in topological 
orders" -> "order".

One of the changes in this patch is that the `jar` tool will locate JMOD 
files on the module path. I assume this is to provide flexibility to 
those creating a modular JAR that want to tie it to a specific JDK 
build. I guess it's okay but I suspect will not be widely used.

-Alan