# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1523103997 -32400
# Node ID 632b928992039afe96df8f99a8dec6127ff983f1
# Parent  bbd240f81ac5e59bdab9060a5743a3073a41e048
procutil: drop unused 'newlines' option from popen*() (API)

It's unlikely for us to use the universal_newlines option.

diff -r bbd240f81ac5 -r 632b92899203 mercurial/utils/procutil.py
--- 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