Merge tag 'upstream/2015.07.21'

[youtubedl] / youtube_dl / postprocessor / common.py

diff --git a/youtube_dl/postprocessor/common.py b/youtube_dl/postprocessor/common.py
index e54ae678da17bef5c5848bc7165d42d5eec912a4..4191d040bb1da468e248d57b78df1afdacf64cd0 100644 (file)
--- a/youtube_dl/postprocessor/common.py
+++ b/youtube_dl/postprocessor/common.py
@@ -1,6 +1,11 @@
 from __future__ import unicode_literals
 
 from __future__ import unicode_literals
 

-from ..utils import PostProcessingError
+import os
+
+from ..utils import (
+    PostProcessingError,
+    encodeFilename,
+)

 
 
 class PostProcessor(object):
 
 
 class PostProcessor(object):


@@ -18,6 +23,9 @@ class PostProcessor(object):
 
     PostProcessor objects follow a "mutual registration" process similar
     to InfoExtractor objects.
 
     PostProcessor objects follow a "mutual registration" process similar
     to InfoExtractor objects.

+
+    Optionally PostProcessor can use a list of additional command-line arguments
+    with self._configuration_args.

     """
 
     _downloader = None
     """
 
     _downloader = None


@@ -37,14 +45,27 @@ class PostProcessor(object):
         one has an extra field called "filepath" that points to the
         downloaded file.
 
         one has an extra field called "filepath" that points to the
         downloaded file.
 

-        This method returns a tuple, the first element of which describes
-        whether the original file should be kept (i.e. not deleted - None for
-        no preference), and the second of which is the updated information.
+        This method returns a tuple, the first element is a list of the files
+        that can be deleted, and the second of which is the updated
+        information.

 
         In addition, this method may raise a PostProcessingError
         exception if post processing fails.
         """
 
         In addition, this method may raise a PostProcessingError
         exception if post processing fails.
         """

-        return None, information  # by default, keep file and do nothing
+        return [], information  # by default, keep file and do nothing
+
+    def try_utime(self, path, atime, mtime, errnote='Cannot update utime of file'):
+        try:
+            os.utime(encodeFilename(path), (atime, mtime))
+        except Exception:
+            self._downloader.report_warning(errnote)
+
+    def _configuration_args(self, default=[]):
+        pp_args = self._downloader.params.get('postprocessor_args')
+        if pp_args is None:
+            return default
+        assert isinstance(pp_args, list)
+        return pp_args

 
 
 class AudioConversionError(PostProcessingError):
 
 
 class AudioConversionError(PostProcessingError):