> 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. > > 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: update jlink plugin.properties to include the new argument ------------- Changes: - all: https://git.openjdk.org/jdk/pull/16066/files - new: https://git.openjdk.org/jdk/pull/16066/files/182ef8b1..510a3ecf Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=16066&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16066&range=00-01 Stats: 9 lines in 1 file changed: 6 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/16066.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/16066/head:pull/16066 PR: https://git.openjdk.org/jdk/pull/16066
