From d7ff9156fb67a98a7159bcfe37343d2acb55dffb Mon Sep 17 00:00:00 2001 From: Roy Nieterau Date: Tue, 20 Feb 2018 17:51:52 +0100 Subject: [PATCH] Refactor how arguments are passed to subprocess.Popen --- .../plugins/fusion/publish/publish_image_sequences.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/colorbleed/plugins/fusion/publish/publish_image_sequences.py b/colorbleed/plugins/fusion/publish/publish_image_sequences.py index 94f5187ffc..fe3413d9a7 100644 --- a/colorbleed/plugins/fusion/publish/publish_image_sequences.py +++ b/colorbleed/plugins/fusion/publish/publish_image_sequences.py @@ -42,16 +42,13 @@ class PublishImageSequence(pyblish.api.Extractor): assert os.path.isfile(path), ("Stored path is not a file for %s" % instance.data["name"]) - # Get the script to execute - script = _get_script() - cmd = 'python {0} --paths "{1}"'.format(script, path) - # Suppress any subprocess console startupinfo = subprocess.STARTUPINFO() startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW startupinfo.wShowWindow = subprocess.SW_HIDE - process = subprocess.Popen(cmd, + process = subprocess.Popen(["python", _get_script(), + "--paths", path], bufsize=1, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, @@ -78,4 +75,3 @@ class PublishImageSequence(pyblish.api.Extractor): valid = bool(context) if not valid: raise RuntimeError("Unable to publish image sequences") -