Few minor jpackage issues:
- Remove debug logging of arguments at [1]. It looks like a leftover from some debug session years ago.
- The default module path is not passed to jlink. A regression fromJDK-8333664. jlink without an explicit "--module-path" option set to the default module path should work just fine, so this issue doesn't have a functional impact.
- Fix incorrect configuration of BuildEnv instance in msi/exe bundlers. The directory for cooking app image when bundling msi/exe packages is supposed to be "${TMP}/image", but it is "${TMP}/image/${AppName}", which is correct for app image bundling, but not for msi/exe bundling. The issue doesn't have a functional impact.
[1] https://github.com/openjdk/jdk/blob/0cb8ccd89a659eaf1e245cfb7f8c32fb16bff4c7/src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java#L128
- Remove debug logging of arguments at [1]. It looks like a leftover from some debug session years ago.
- The default module path is not passed to jlink. A regression from
- Fix incorrect configuration of BuildEnv instance in msi/exe bundlers. The directory for cooking app image when bundling msi/exe packages is supposed to be "${TMP}/image", but it is "${TMP}/image/${AppName}", which is correct for app image bundling, but not for msi/exe bundling. The issue doesn't have a functional impact.
[1] https://github.com/openjdk/jdk/blob/0cb8ccd89a659eaf1e245cfb7f8c32fb16bff4c7/src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java#L128
- links to
-
Commit(master)
openjdk/jdk/b0af41d6
-
Review(master)
openjdk/jdk25u/357
-
Review(master)
openjdk/jdk/27876