RFR: JDK-8217792 : Investigate what modules are included

[Mandy Chung]

On 1/30/19 2:05 PM, Andy Herrick wrote:
> Please review the jpackage fix for bug [1] at [2].
> 
> This is a fix for the JDK-8200758-branch branch of the open sandbox 
> repository (jpackage).
> 
> JDK-8217792 : Investigate what modules are included
> 
> For modules included in the runtime of a non-modular application, we now 
> computes all modules that export an API as the root modules and compute 
> their transitive closure, and do service binding.  As a result, by 
> default you get all modules except 10 internal modules.
> 
> For jre-installer (without --runtime-image option) we revert to 
> including all modules.
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8217792
> 
> [2] http://cr.openjdk.java.net/~herrick/8217792/webrev.03/

  368         ModuleFinder finder = ModuleFinder.ofSystem();

This means that you assume that the JMOD files used for linking
is the same version as the system module path.  This assumption
is fine since jlink currently does not support linking a different
version.

You may want to add a comment if it helps.

Mandy