changeset 37464:632b92899203

procutil: drop unused 'newlines' option from popen*() (API) It's unlikely for us to use the universal_newlines option.
author Yuya Nishihara <yuya@tcha.org>
date Sat, 07 Apr 2018 21:26:37 +0900
parents bbd240f81ac5
children 39e5e346eba7
files mercurial/utils/procutil.py
diffstat 1 files changed, 4 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/utils/procutil.py	Sat Apr 07 21:23:42 2018 +0900
+++ b/mercurial/utils/procutil.py	Sat Apr 07 21:26:37 2018 +0900
@@ -128,27 +128,25 @@
                          stdin=subprocess.PIPE)
     return _pfile(p, p.stdin)
 
-def popen2(cmd, env=None, newlines=False):
+def popen2(cmd, env=None):
     # Setting bufsize to -1 lets the system decide the buffer size.
     # The default for bufsize is 0, meaning unbuffered. This leads to
     # poor performance on Mac OS X: http://bugs.python.org/issue4194
     p = subprocess.Popen(cmd, shell=True, bufsize=-1,
                          close_fds=closefds,
                          stdin=subprocess.PIPE, stdout=subprocess.PIPE,
-                         universal_newlines=newlines,
                          env=env)
     return p.stdin, p.stdout
 
-def popen3(cmd, env=None, newlines=False):
-    stdin, stdout, stderr, p = popen4(cmd, env, newlines)
+def popen3(cmd, env=None):
+    stdin, stdout, stderr, p = popen4(cmd, env)
     return stdin, stdout, stderr
 
-def popen4(cmd, env=None, newlines=False, bufsize=-1):
+def popen4(cmd, env=None, bufsize=-1):
     p = subprocess.Popen(cmd, shell=True, bufsize=bufsize,
                          close_fds=closefds,
                          stdin=subprocess.PIPE, stdout=subprocess.PIPE,
                          stderr=subprocess.PIPE,
-                         universal_newlines=newlines,
                          env=env)
     return p.stdin, p.stdout, p.stderr, p