On Tue, 13 Jan 2026 16:37:44 GMT, Alexey Semenyuk <[email protected]> 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

Reply via email to