RFR: 8368192: Test java/lang/ProcessBuilder/Basic.java#id0 fails with Exception: Stack trace [v3]
Thomas Stuefe
stuefe at openjdk.org
Tue Sep 23 14:47:07 UTC 2025
On Tue, 23 Sep 2025 14:08:35 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> 8368192: Test java/lang/ProcessBuilder/Basic.java#id0 fails with Exception: Stack trace
>> With MacOsX Sequoia 15.7 the command environment contains a new TMPDIR variable.
>> The ProcessBuild/Basic test checks for a small set of variables set in the parent
>> and ignores variables that are set by the OS or runtime, now including TMPDIR.
>>
>> In the Basic test, for MacOSX, checks of the environment in the child should ignore TMPDIR in addition
>> to the other existing environment variables that are expected.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>
> Simplify the code to match and remove expected environment variables on MacOSX.
Looks good. Small suggestion, up to you if you take it.
test/jdk/java/lang/ProcessBuilder/Basic.java line 801:
> 799: // Check and remove TMPDIR
> 800: cleanedVars = matchAndRemove(cleanedVars,
> 801: "TMPDIR=[^,]*,");
Why not just use reluctant match? As in `"TMPDIR=.*?,"`
-------------
Marked as reviewed by stuefe (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27434#pullrequestreview-3258362224
PR Review Comment: https://git.openjdk.org/jdk/pull/27434#discussion_r2372569169
More information about the core-libs-dev
mailing list