Merge pull request #1440 from Bastian-Krause/bst/externalconsole-bufsize

jluebbe · web-flow · commit 0d293be5e69a · 2024-07-17T09:09:49.000+02:00
driver/externalconsoledriver: use bufsize=0 for opening external process
diff --git a/labgrid/driver/externalconsoledriver.py b/labgrid/driver/externalconsoledriver.py
@@ -33,7 +33,7 @@ def open(self):
             return
         cmd = shlex.split(self.cmd)
         self._child = subprocess.Popen(
-            cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=None
+            cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=None, bufsize=0
         )
 
         # make stdout non-blocking

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ def open(self):`
`33`	`33`	`return`
`34`	`34`	`cmd = shlex.split(self.cmd)`
`35`	`35`	`self._child = subprocess.Popen(`
`36`		`- cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=None`
	`36`	`+ cmd, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=None, bufsize=0`
`37`	`37`	`)`
`38`	`38`
`39`	`39`	`# make stdout non-blocking`