From 7afb24a972ab296b86da048bd6f387231e5a9197 Mon Sep 17 00:00:00 2001
From: anseki <AnSeki.aff@gmail.com>
Date: Sun, 29 Jun 2014 07:00:15 +0900
Subject: [PATCH] Add reading shell through files (pipe).

---
 lib/read.bat         | 12 ++++----
 lib/readline-sync.js | 67 +++++++++++++++++++++++++++++++++++++-------
 package.json         |  5 +---
 3 files changed, 64 insertions(+), 20 deletions(-)

diff --git a/lib/read.bat b/lib/read.bat
index 942cfba..9d2b9e6 100644
--- a/lib/read.bat
+++ b/lib/read.bat
@@ -1,6 +1,6 @@
-@ECHO OFF
-SETLOCAL
-SET /P LINE=
-SET /P DUM=%LINE%<NUL
-ENDLOCAL
-EXIT /B 0
+@echo off
+setlocal
+set /p LINE=<CON >CON
+set /p DUM="%LINE%"<NUL
+endlocal
+exit /b 0
diff --git a/lib/readline-sync.js b/lib/readline-sync.js
index 1d1283f..649b7be 100644
--- a/lib/readline-sync.js
+++ b/lib/readline-sync.js
@@ -15,16 +15,7 @@ var promptText = '> ',
   stdin = process.stdin,
   stdout = process.stdout,
   buffer = new Buffer(BUF_SIZE),
-  useShell = true;
-
-function _readlineShell() {
-      // Win isn't supported by sync-exec v0.3.2
-  var command = /* require('os').platform() === 'win32' ?
-      'cmd "' + __dirname + '\\read.bat' : */
-      'sh "' + __dirname + '/read.sh"',
-    resExec = require('sync-exec')(command); // instead of execSync (node v0.12+)
-  return resExec.status === 0 && !resExec.stderr ? (resExec.stdout + '') : false;
-}
+  useShell = true, tempdir;
 
 function _readlineSync(display) {
   var input = '', rsize, err;
@@ -66,6 +57,62 @@ function _readlineSync(display) {
   return input.trim();
 }
 
+function _readlineShell() {
+  // piping via files instead of execSync (node v0.12+)
+  var shellPath, args, shellStdout,
+    pathStdout = getTempfile('readline-sync.stdout'),
+    pathStatus = getTempfile('readline-sync.status'),
+    pathDone   = getTempfile('readline-sync.done');
+
+  if (process.platform === 'win32') {
+    // The quote (") is escaped by node before parsed by shell. Then use ENV{Q}.
+    shellPath = 'cmd.exe';
+    args = ['/V:ON', '/S', '/C', '%Q%' + __dirname + '\\read.bat%Q% >%Q%' + pathStdout +
+      '%Q% & (ECHO !ERRORLEVEL!)>%Q%' + pathStatus + '%Q% & (ECHO 1)>%Q%' + pathDone + '%Q%'];
+  } else {
+    shellPath = '/bin/sh';
+    args = ['-c', '(' + shellPath + ' "' + __dirname + '/read.sh") >"' + pathStdout +
+      '"; echo $? >"' + pathStatus + '"; echo 1 >"' + pathDone + '"'];
+  }
+
+  stdin.pause(); // re-start in child process
+  require('child_process').execFile(shellPath, args, {env: {Q: '"'}});
+
+  while (true) {
+    if (fs.readFileSync(pathDone, {encoding: encoding}).trim() === '1') { break; }
+  }
+  if (fs.readFileSync(pathStatus, {encoding: encoding}).trim() === '0') {
+    shellStdout = fs.readFileSync(pathStdout, {encoding: encoding});
+  }
+
+  fs.unlinkSync(pathStdout);
+  fs.unlinkSync(pathStatus);
+  fs.unlinkSync(pathDone);
+  return shellStdout;
+}
+
+function getTempfile(name) {
+  var path = require('path'), filepath, suffix = '', fd;
+  tempdir = tempdir || require('os').tmpdir();
+
+  while (true) {
+    filepath = path.join(tempdir, name + suffix);
+    try {
+      fd = fs.openSync(filepath, 'wx');
+    } catch (e) {
+      if (e.code === 'EEXIST') {
+        suffix++;
+        continue;
+      } else {
+        throw e;
+      }
+    }
+    fs.closeSync(fd);
+    break;
+  }
+  return filepath;
+}
+
 // for dev
 exports.useShellSet = function(use) { useShell = use; };
 
diff --git a/package.json b/package.json
index 1a8c30a..f3653b4 100644
--- a/package.json
+++ b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "readline-sync",
   "description": "Synchronous Readline",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "homepage": "https://github.com/anseki/readline-sync",
   "author": {
     "name": "anseki"
@@ -23,9 +23,6 @@
   "engines": {
     "node": ">= 0.8.0"
   },
-  "dependencies": {
-    "sync-exec": "~0.3.2"
-  },
   "keywords": [
     "readline",
     "synchronous",