0
0
mirror of https://github.com/nodejs/node.git synced 2024-11-29 23:16:30 +01:00
nodejs/test/pseudo-tty
Ben Noordhuis 9c1b18a59f tty: fix 'resize' event regression
It's not wholly clear what commit introduced the regression but between
v8.4.0 and v8.5.0 the 'resize' event stopped getting emitted when the
tty was resized.

The SIGWINCH event listener apparently was being installed before the
support code for `process.on('SIGWINCH', ...)` was.  Fix that by moving
said support code to real early in the bootstrap process.

This commit also seems to fix a Windows-only "write EINVAL" error for
reasons even less well-understood...

Fixes: https://github.com/nodejs/node/issues/16141
Fixes: https://github.com/nodejs/node/issues/16194
PR-URL: https://github.com/nodejs/node/pull/16225
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
2017-11-15 12:18:58 +01:00
..
no_dropped_stdio.js
no_dropped_stdio.out
no_interleaved_stdio.js
no_interleaved_stdio.out
pseudo-tty.status tty: fix 'resize' event regression 2017-11-15 12:18:58 +01:00
ref_keeps_node_running.js
ref_keeps_node_running.out
stdin-setrawmode.js
stdin-setrawmode.out
test-handle-wrap-isrefed-tty.js
test-handle-wrap-isrefed-tty.out
test-stderr-stdout-handle-sigwinch.js
test-stderr-stdout-handle-sigwinch.out
test-tty-isatty.js
test-tty-isatty.out
test-tty-stdout-end.js
test-tty-stdout-end.out
test-tty-stdout-resize.js tty: fix 'resize' event regression 2017-11-15 12:18:58 +01:00
test-tty-stdout-resize.out tty: fix 'resize' event regression 2017-11-15 12:18:58 +01:00
test-tty-stream-constructors.js
test-tty-stream-constructors.out
test-tty-window-size.js
test-tty-window-size.out
test-tty-wrap.js
test-tty-wrap.out
testcfg.py