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