RFR: 8317634: Introduce the ability to exclude modules from dedup-legal-notices plugin of jlink [v3]

[Jaikiran Pai]

On Fri, 6 Oct 2023 02:22:52 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this change which proposes to implement the enhancement request noted in https://bugs.openjdk.org/browse/JDK-8317634?
>> 
>> The change in this commit introduces a new `exclude-modules` argument to the `dedup-legal-notices` jlink plugin. This optional argument takes a comma separated values of module names. These module names are then excluded from the de-duplication of license notice files during image generation.
>> 
>> (Edited to add some context) The motivation for this change is discussed here https://github.com/openjdk/jdk/pull/13686#issuecomment-1747301865
>> 
>> Two new test methods have been added to the existing `LegalFilePluginTest` jtreg test case to verify this implementation. Existing tests in `test/jdk/tools/jlink` continue to pass. tier testing is currently in progress.
>
> Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:
> 
>   minor changes to the text in plugin.properties for the new argument, to render better on command line

Thank you for that input Mandy. Given that this enhancement was requested specifically for trying to solve the javadoc generation issue in https://github.com/openjdk/jdk/pull/13686 and since this doesn't solve it, I'll go ahead and close this enhancement request.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16066#issuecomment-1770587347