> Added a test case to JLinkOptionsTest to test that jpackage doesn't bind > services in app's runtime by default. > > The test builds two app images for the same app. One with the default jlink > options and another with `--bind-services` jlink option. The test compares > module lists from both runtimes. The expected result is that the runtime > built with `--bind-services` jlink option has additional modules. > > The PR must be integrated after https://github.com/openjdk/jdk/pull/22644. > Otherwise, the newly added test will fail.
Alexey Semenyuk has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision: - Merge branch 'master' into JDK-8346434 - Use HelloApp.assertApp() to run a launcher - Merge branch 'master' into JDK-8346434 - Trailing whitespace removed - Merge branch 'master' into JDK-8346434 - Copyright year updated - 8346434: Add test for non-automatic service binding ------------- Changes: - all: https://git.openjdk.org/jdk/pull/23047/files - new: https://git.openjdk.org/jdk/pull/23047/files/284c1439..6d6e0442 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=23047&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23047&range=01-02 Stats: 5695 lines in 298 files changed: 1949 ins; 2552 del; 1194 mod Patch: https://git.openjdk.org/jdk/pull/23047.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/23047/head:pull/23047 PR: https://git.openjdk.org/jdk/pull/23047
