From 59b869e83123b7ed8dc678bc76dc80e2fbb5256f Mon Sep 17 00:00:00 2001 From: Myles Borins Date: Tue, 28 Mar 2017 04:50:24 -0400 Subject: [PATCH] Revert "Revert "repl: disable Ctrl+C support..." Full original message: Revert "repl: disable Ctrl+C support on win32 for now" The original fix was a stop gap until a libuv update landed. As the libuv update has not yet landed on v6.x the revert should not have landed. This commit reverts 1d400ea484 reapplying the stopgap fix until we update libuv. Fixes: https://github.com/nodejs/node/issues/12085 Refs: https://github.com/nodejs/node/pull/8645 PR-URL: https://github.com/nodejs/node/pull/12123 Reviewed-By: Anna Henningsen --- lib/repl.js | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/lib/repl.js b/lib/repl.js index 52bb1b6b53986c..049977ce995f34 100644 --- a/lib/repl.js +++ b/lib/repl.js @@ -322,7 +322,13 @@ function REPLServer(prompt, if (!err) { // Unset raw mode during evaluation so that Ctrl+C raises a signal. let previouslyInRawMode; - if (self.breakEvalOnSigint) { + + // Temporarily disabled on Windows due to output problems that likely + // result from the raw mode switches here, see + // https://github.com/nodejs/node/issues/7837 + // Setting NODE_REPL_CTRLC is meant as a temporary opt-in for debugging. + if (self.breakEvalOnSigint && + (process.platform !== 'win32' || process.env.NODE_REPL_CTRLC)) { // Start the SIGINT watchdog before entering raw mode so that a very // quick Ctrl+C doesn’t lead to aborting the process completely. utilBinding.startSigintWatchdog(); @@ -342,7 +348,8 @@ function REPLServer(prompt, result = script.runInContext(context, scriptOptions); } } finally { - if (self.breakEvalOnSigint) { + if (self.breakEvalOnSigint && + (process.platform !== 'win32' || process.env.NODE_REPL_CTRLC)) { // Reset terminal mode to its previous value. self._setRawMode(previouslyInRawMode);