chgserver: update comment describing when to fall back to core _runsystem() stable
authorYuya Nishihara <yuya@tcha.org>
Wed, 26 Sep 2018 21:21:05 +0900
branchstable
changeset 38800 1b9be0b26511
parent 38799 e5724be689b3
child 38801 23a00bc90a3c
chgserver: update comment describing when to fall back to core _runsystem() The condition "output needs to be captured" was moved to the core ui, and removed at fbce78c58f1e "chg: refactor ui.system() to be partly overridden." The next patch will add one more "if" to address the issue 5992.
mercurial/chgserver.py
--- a/mercurial/chgserver.py	Wed Sep 26 20:53:59 2018 +0900
+++ b/mercurial/chgserver.py	Wed Sep 26 21:21:05 2018 +0900
@@ -198,10 +198,10 @@
                 self._csystem = csystem
 
         def _runsystem(self, cmd, environ, cwd, out):
-            # fallback to the original system method if the output needs to be
-            # captured (to self._buffers), or the output stream is not stdout
-            # (e.g. stderr, cStringIO), because the chg client is not aware of
-            # these situations and will behave differently (write to stdout).
+            # fallback to the original system method if
+            #  a. the output stream is not stdout (e.g. stderr, cStringIO),
+            # because the chg client is not aware of these situations and
+            # will behave differently (i.e. write to stdout).
             if (out is not self.fout
                 or not util.safehasattr(self.fout, 'fileno')
                 or self.fout.fileno() != procutil.stdout.fileno()):