patch on issue #5

removed pipe on stdin and stderr to leave only stdout, this seems to be fine now
PyMoDAQ · May 11, 2021 · ef15fc9 · ef15fc9 · Semptum · Feb 21, 2024
1 parent ca205dc
commit ef15fc9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/pymodaq_plugin_manager/data/PluginList.json b/src/pymodaq_plugin_manager/data/PluginList.json
@@ -1,6 +1,6 @@
 {
 	"name": "pymodaq-pluginList",
-	"version": "0.0.13",
+	"version": "0.0.14",
 	"pymodaq-plugins": [
 		{
 			"plugin-name": "pymodaq_plugins",

diff --git a/src/pymodaq_plugin_manager/manager.py b/src/pymodaq_plugin_manager/manager.py
@@ -287,8 +287,11 @@ def do_action(self):
 
     def do_subprocess(self, command):
         try:
-            with subprocess.Popen(command, stdout=subprocess.PIPE, stdin=sys.stdout, stderr=sys.stdout,
-                                  universal_newlines=True) as sp:
+            self.info_widget.moveCursor(QTextCursor.End)
+            self.info_widget.insertPlainText(' '.join(command))
+            self.info_widget.moveCursor(QTextCursor.End)
+
+            with subprocess.Popen(command, stdout=subprocess.PIPE, universal_newlines=True, shell=True) as sp:
                 while True:
                     self.info_widget.moveCursor(QTextCursor.End)
                     self.info_widget.insertPlainText(sp.stdout.readline())