Review Request: JDK-8174826 jlink does not provide support for linking in service provider modules

[Alan Bateman]

On 22/03/2017 19:11, Mandy Chung wrote:

> Webrev:
>    http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8174826/webrev.00/
>
> This is a proposal to resolve the open issue listed in JEP 282
> about jlink and service binding.
The output for --suggest-providers looks good.

For --verbose then the "Providers: " list looks okay when used in 
conjunction with --bind-services. I would be tempted to leave it out 
when --bind-services is not specified because there is no service binding.

A few random comments:

- JImageTask is now using toUpperCase() which is locale specific and 
should be changed.

- Jlink.moduleFinder() unconditionally uses the runtime version. Not a 
bug introduced by your changes but this method really needs to locate 
java.base and use its version when creating the module path. We should 
probably create an issue for that and fix it another time.

- JlinkTask.uses then the stream() isn't needed as Set defines forEach.

-Alan