[Erp5-report] r45525 priscila.manhaes - /erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloud...

nobody at svn.erp5.org nobody at svn.erp5.org
Fri Apr 15 17:17:53 CEST 2011


Author: priscila.manhaes
Date: Fri Apr 15 17:17:53 2011
New Revision: 45525

URL: http://svn.erp5.org?rev=45525&view=rev
Log:
Adding logger error for convert method

Modified:
    erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py

Modified: erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py
URL: http://svn.erp5.org/erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py?rev=45525&r1=45524&r2=45525&view=diff
==============================================================================
--- erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py [utf8] (original)
+++ erp5/trunk/utils/cloudooo.handler.ffmpeg/src/cloudooo/handler/ffmpeg/handler.py [utf8] Fri Apr 15 17:17:53 2011
@@ -29,6 +29,7 @@
 from zope.interface import implements
 from cloudooo.interfaces.handler import IHandler
 from cloudooo.file import File
+from cloudooo.util.util import logger
 from subprocess import Popen, PIPE
 from tempfile import mktemp
 
@@ -62,9 +63,9 @@ class Handler(object):
                "-y",
                output_url]
     # XXX ffmpeg has a bug that needs this options to work with webm format
-    if destination_format == "webm":
-      command.insert(3, "32k")
-      command.insert(3, "-ab")
+#    if destination_format == "webm":
+#      command.insert(3, "32k")
+#      command.insert(3, "-ab")
     try:
       stdout, stderr = Popen(command,
                              stdout=PIPE,
@@ -72,6 +73,8 @@ class Handler(object):
                              close_fds=True,
                              env=self.environment).communicate()
       self.input.reload(output_url)
+      if len(self.input.getContent()) == 0:
+        logger.error(stderr.split("\n")[-2])
       return self.input.getContent()
     finally:
       self.input.trash()



More information about the Erp5-report mailing list