RFR: 8356047: [macos] jpackage produces confusing post- and pre- installation PKG scripts [v8]
Alexey Semenyuk
asemenyuk at openjdk.org
Wed Oct 8 03:01:08 UTC 2025
On Wed, 8 Oct 2025 02:36:13 GMT, Alexander Matveev <almatvee at openjdk.org> wrote:
>> Let's address customizable resource logging in a separate patch. I filed [JDK-8369334](https://bugs.openjdk.org/browse/JDK-8369334) for this effort.
>
> I just fix it. Let me know what you think. I can revert my last commit.
It fixes one specific issue that just popped up. There may be more, but jpackage tests don't trap "null"-s in log messages emitted by OverridableResource.
The change to OverridableResource is wrong for a few reasons:
- It unconditionally writes a log message; log messages in OverridableResource are created only when the destination file or output stream is not `null`. This change contradicts logging in OverridableResource.
- It assumes that the set of resource sources is fixed. This assumption is wrong. It is possible to configure the order and the set of resource sources, see `OverridableResource.setSourceOrder()`.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25510#discussion_r2412381007
More information about the core-libs-dev
mailing list