On Mon, 6 May 2024 08:28:50 GMT, Jan Lahoda <jlah...@openjdk.org> wrote:
> When integrating: > https://github.com/openjdk/jdk/pull/18996 > > I've forgot to push one last commit which was stabilizing the test of Mac > OS/X. I am sorry for that. > > The test will create a pseudo terminal, and change the current process > stdin/stdout to write into the pty. But, there's nothing reading from the pty > on the other side. This mostly works, OK, but the `ProcessTools` will write a > debug log into `System.out` (which will write to stdout, which is the pty), > and that blocks on Mac OS/X, presumably because there's nothing reading from > the pty. > > This patch changes the `System.out` for the main process to a scratch value, > so no write to stdout/FD 1 is done, and no blocking should happen. (The > process' stdout/FD 1 should remain attached to the pty.) This pull request has now been integrated. Changeset: 15862a2f Author: Jan Lahoda <jlah...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/15862a2f116331b7f439619c3aa1b5965e737044 Stats: 8 lines in 1 file changed: 6 ins; 1 del; 1 mod 8331708: jdk/internal/jline/RedirectedStdOut.java times-out on macosx-aarch64 Reviewed-by: asotona ------------- PR: https://git.openjdk.org/jdk/pull/19097
