RFR: JDK-8219536: Add Option for user defined jlink options

[Alexey Semenyuk]

http://cr.openjdk.java.net/~herrick/8219536/webrev.03/test/jdk/tools/jpackage/helpers/jdk/jpackage/test/JPackageCommand.java.sdiff.html:731 
- 'launcherName' parameter of readRuntimeReleaseFile() function seems to 
be not used.

http://cr.openjdk.java.net/~herrick/8219536/webrev.03/test/jdk/tools/jpackage/share/jdk/jpackage/tests/JLinkOptionsTest.java.html: 
copyright year should be 2020.

http://cr.openjdk.java.net/~herrick/8219536/webrev.03/test/jdk/tools/jpackage/share/jdk/jpackage/tests/JLinkOptionsTest.java.html: 
128 - There is no point to assert if 'cfgfile' is not null. It should be 
always non null for application packaging or readLaunherCfgFile.

For testing if string contains some values, I'd suggest to use 
TKit.assertTextStream():
---
List<String> mods = List.of(release.get(1));
if (required != null) {
     for (String s : required) {
         TKit.assertTextStream(s).label("mods").apply(mods.stream());
     }
}

if (prohibited != null) {
     for (String s : prohibited) {
TKit.assertTextStream(s).label("mods").negate().apply(mods.stream());
     }
}
---
Will save you from maintaining explicit log messages.

- Alexey

On 4/23/2020 10:08 AM, Andy Herrick wrote:
> Please review webrev at [1] to address issue [2].
>
> This is the new feature to add the jpackage option --jlink-options as 
> specified in CSR at [3]
>
> /Andy
>
>
> [1] http://cr.openjdk.java.net/~herrick/8219536/webrev.03
>
> [2] https://bugs.openjdk.java.net/browse/JDK-8219536
>
> [3] https://bugs.openjdk.java.net/browse/JDK-8243272
>