RFR: 8375242: [macos] Improve jpackage signing coverage

Alexey Semenyuk asemenyuk at openjdk.org
Fri Jan 16 04:21:07 UTC 2026


On Fri, 16 Jan 2026 02:03:03 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:

>  I think we need ability to pass additional arguments to setFakeRuntime(String version)

To suit the needs of a single test? Any chance we need a fake runtime with the "release" file in other tests?

> In this case we will have `createInputRuntimeImage(RuntimeImageType role, String version)`

Two major problems with this approach:
 - Eventually, we will end up with `createInputRuntimeImage(RuntimeImageType role, String version, String fooProperty, boolean barProperty, Date dateProperty)` signature. Which will be unreadable, just like we had it at https://github.com/openjdk/jdk/blob/6c48f4ed707bf0b15f9b6098de30db8aae6fa40f/src/jdk.jpackage/share/classes/jdk/jpackage/internal/IOUtils.java#L108 The calls will be equally unreadable to: https://github.com/openjdk/jdk/blob/6c48f4ed707bf0b15f9b6098de30db8aae6fa40f/src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacDmgPackager.java#L330
 - When the external runtime is specified for the test with the "jdk.jpackage.runtime-image" system property, what to do with the "version" parameter?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29205#discussion_r2696835672


More information about the core-libs-dev mailing list