On Wed, 16 Sep 2026 12:38:06 GMT, Jaikiran Pai <[email protected]> wrote:
>> Daniel Jeliński has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Print expected output > > test/jdk/java/lang/ProcessBuilder/Basic.java line 1974: > >> 1972: : "LC_ALL=C,"), >> 1973: "Incorrect handling of envstrings containing NULs, "+ >> 1974: "got: >'" + commandOutput + "'<"); > > Hello Daniel, given that `systemRoot` and `libpath` are host dependent and we > don't seem to print them anywhere in this test, could we also include the > values of those 2 somewhere in this exception or at least as a separate > message some place in this test? That might help why the equality check fails. Good point. Updated. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32902#discussion_r4026489324
