RFR: 8327095: (ch) java/nio/channels/AsyncCloseAndInterrupt.java: improve error message when mkfifo fails [v3]
Brian Burkhalter
bpb at openjdk.org
Tue Mar 12 16:35:14 UTC 2024
On Tue, 12 Mar 2024 16:20:26 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
>> test/jdk/java/nio/channels/AsyncCloseAndInterrupt.java line 144:
>>
>>> 142: OutputAnalyzer oa = ProcessTools.executeProcess(pb);
>>> 143: oa.waitFor();
>>> 144: if (oa.pid() != 0)
>>
>> The test is currently failing for me on WSL1 (Linux configuration), both with and without your changes; presumably NTFS doesn't support FIFO files. Should we fail or skip the test in this case? I'm fine with either, WSL is not a supported build config, and this is not the only failure.
>>
>> If you choose to fail the test, then you can use:
>>
>> ProcessTools.executeProcess(pb).shouldHaveExitValue(0);
>>
>> this will dump the output and throw a RuntimeException for you.
>>
>> If you choose to skip the fifo test instead, `oa.getExitValue()` is what you need here.
>
> The `FileStore` type is "NTFS"?
> `oa.getExitValue()` is what you need here.
Using `oa.pid()` was lame.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18072#discussion_r1521787628
More information about the nio-dev
mailing list