RFR: 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts [v6]
Alexander Matveev
almatvee at openjdk.org
Fri Oct 3 21:12:35 UTC 2025
> - Removed pre- and post- installation PKG scripts.
> - This code is not needed and PKG should create destination folder and set correct permissions.
> - If for some reason it is not happens, permissions issues should be fixed when jpackage prepares application bundle. PKG should keep all permissions unchanged when packaging and installing bundle.
> - Users will have ability to provide pre- and post- installation PKG scripts if needed.
> - `INSTALL_LOCATION` and `APP_LOCATION` substitution is removed, since `$2` argument in scripts is same as `INSTALL_LOCATION`.
> - I think code in these scripts are some legacy leftovers.
> - Added test to test that pre- and post-scripts are no longer exist and can be added via `--resource-dir`.
Alexander Matveev 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 remote-tracking branch 'upstream/master' into JDK-8356047
- 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts [v4]
- 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts [v3]
- Merge remote-tracking branch 'upstream/master' into JDK-8356047
- 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts [v2]
- Merge remote-tracking branch 'upstream/master' into JDK-8356047
- 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/25510/files
- new: https://git.openjdk.org/jdk/pull/25510/files/38da308c..17d948c2
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=25510&range=05
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=25510&range=04-05
Stats: 11234 lines in 552 files changed: 5424 ins; 2593 del; 3217 mod
Patch: https://git.openjdk.org/jdk/pull/25510.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/25510/head:pull/25510
PR: https://git.openjdk.org/jdk/pull/25510
More information about the core-libs-dev
mailing list