From 5cf11dd64aab79790c4a0bec6df5e55387dcfb1c Mon Sep 17 00:00:00 2001
From: Toke Jepsen <tokejepsen@gmail.com>
Date: Wed, 24 Jul 2019 23:38:34 +0100
Subject: [PATCH] Log output from ffmpeg process.

---
 pype/plugins/global/publish/extract_review.py | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/pype/plugins/global/publish/extract_review.py b/pype/plugins/global/publish/extract_review.py
index 3a764b19c3..1b66b4e9d2 100644
--- a/pype/plugins/global/publish/extract_review.py
+++ b/pype/plugins/global/publish/extract_review.py
@@ -136,12 +136,21 @@ class ExtractReview(pyblish.api.InstancePlugin):
 
                             # run subprocess
                             self.log.debug("{}".format(subprcs_cmd))
-                            sub_proc = subprocess.Popen(subprcs_cmd)
-                            sub_proc.wait()
+                            sub_proc = subprocess.Popen(
+                                subprcs_cmd,
+                                stdout=subprocess.PIPE,
+                                stderr=subprocess.STDOUT,
+                                stdin=subprocess.PIPE,
+                                cwd=os.path.dirname(output_args[-1])
+                            )
+
+                            output = sub_proc.communicate()[0]
 
                             if not os.path.isfile(full_output_path):
-                                raise FileExistsError(
-                                    "Quicktime wasn't created succesfully")
+                                raise ValueError(
+                                    "Quicktime wasn't created succesfully: "
+                                    "{}".format(output)
+                                )
 
                             # create representation data
                             repre_new.update({