RFR: 8375061: Multiple jpackage tool providers may share the same logging config [v3]
Alexander Matveev
almatvee at openjdk.org
Tue Jan 13 22:32:54 UTC 2026
On Tue, 13 Jan 2026 16:37:44 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> The patch addresses two issues:
>> - Each jpackage tool provider instance should have its own logging config instance.
>> - The test lib should be able to keep output from concurrently running tests isolated from each other.
>>
>> These two fixes are bundled together because it is impossible to verify the jpackage implementation fix without fixing the test lib.
>>
>> The test lib fix replaces all inheritable thread-local variables with scoped values. It fixes the interleaved output issue and simplifies the implementation.
>>
>> Supplementary changes:
>> - jdk.jpackage.internal.cli.Main: when creating a `java.io.PrintWriter` from a `java.io.PrintStream`, copy the charset instead of using the default one.
>
> Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision:
>
> BasicTest: rollback accidental change.
Latest changes looks good.
-------------
Marked as reviewed by almatvee (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/29175#pullrequestreview-3658183144
More information about the core-libs-dev
mailing list