Integrated: 8374216: Assorted changes to jpackage without functional impact

Alexey Semenyuk asemenyuk at openjdk.org
Tue Dec 23 04:42:08 UTC 2025


On Mon, 22 Dec 2025 20:09:13 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:

> Miscellaneous refactorings without functional changes:
> 
> - Replace StringBuffer with StringBuilder where appropriate
> - LauncherBuilder: set reasonable defaults to "defaultIconResourceName" property; move `defaultIconResourceName()` from LauncherFromOptions
> - Make `MacHelper.createRuntimeBundle()` use PackageTest to create a runtime bundle. Will use DMG by default and optionally PKG packaging to create a runtime bundle
> - Remove redundat parameter from `SetBuilder.build()`
> - Make `JUnitUtils.assertArrayEquals()` permit null-s
> - Add `JPackageCommand.withToolProvider()`
> - Make some handy package-private methods in StandardBundlingOperation class public
> - BundleSpec: add "sealed" modifier
> - IdentityWrapper: add `wrapIdentity()`
> - Simplify `DefaultBundlingEnvironment.createNativePackage()`
> - TKit: remove "throws IOException" from `createDirectories()`; Replace `TKit.createDirectories()` with `Files.createDirectories()` where directory clean-up is not used

This pull request has now been integrated.

Changeset: a0094f52
Author:    Alexey Semenyuk <asemenyuk at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a0094f529a6cf7e1e28a20d5033a9a1405f49d9f
Stats:     248 lines in 27 files changed: 127 ins; 41 del; 80 mod

8374216: Assorted changes to jpackage without functional impact

Reviewed-by: almatvee

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

PR: https://git.openjdk.org/jdk/pull/28957


More information about the core-libs-dev mailing list