diff options
| -rw-r--r-- | youtube_dl/extractor/francetv.py | 22 | ||||
| -rw-r--r-- | youtube_dl/postprocessor/ffmpeg.py | 3 | ||||
| -rw-r--r-- | youtube_dl/utils.py | 10 | 
3 files changed, 29 insertions, 6 deletions
| diff --git a/youtube_dl/extractor/francetv.py b/youtube_dl/extractor/francetv.py index 51eb97b2f..f3e0f38b7 100644 --- a/youtube_dl/extractor/francetv.py +++ b/youtube_dl/extractor/francetv.py @@ -48,24 +48,36 @@ class PluzzIE(FranceTVBaseInfoExtractor):  class FranceTvInfoIE(FranceTVBaseInfoExtractor):      IE_NAME = 'francetvinfo.fr' -    _VALID_URL = r'https?://www\.francetvinfo\.fr/replay.*/(?P<title>.+)\.html' +    _VALID_URL = r'https?://www\.francetvinfo\.fr/.*/(?P<title>.+)\.html' -    _TEST = { +    _TESTS = [{          'url': 'http://www.francetvinfo.fr/replay-jt/france-3/soir-3/jt-grand-soir-3-lundi-26-aout-2013_393427.html', -        'file': '84981923.mp4',          'info_dict': { +            'id': '84981923', +            'ext': 'mp4',              'title': 'Soir 3',          },          'params': {              'skip_download': True,          }, -    } +    }, { +        'url': 'http://www.francetvinfo.fr/elections/europeennes/direct-europeennes-regardez-le-debat-entre-les-candidats-a-la-presidence-de-la-commission_600639.html', +        'info_dict': { +            'id': 'EV_20019', +            'ext': 'mp4', +            'title': 'Débat des candidats à la Commission européenne', +            'description': 'Débat des candidats à la Commission européenne', +        }, +        'params': { +            'skip_download': 'HLS (reqires ffmpeg)' +        } +    }]      def _real_extract(self, url):          mobj = re.match(self._VALID_URL, url)          page_title = mobj.group('title')          webpage = self._download_webpage(url, page_title) -        video_id = self._search_regex(r'id-video=(\d+?)[@"]', webpage, 'video id') +        video_id = self._search_regex(r'id-video=((?:[^0-9]*?_)?[0-9]+)[@"]', webpage, 'video id')          return self._extract_video(video_id) diff --git a/youtube_dl/postprocessor/ffmpeg.py b/youtube_dl/postprocessor/ffmpeg.py index 602e370f4..45328ed43 100644 --- a/youtube_dl/postprocessor/ffmpeg.py +++ b/youtube_dl/postprocessor/ffmpeg.py @@ -9,6 +9,7 @@ from .common import AudioConversionError, PostProcessor  from ..utils import (      check_executable,      compat_subprocess_get_DEVNULL, +    encodeArgument,      encodeFilename,      PostProcessingError,      prepend_extension, @@ -48,7 +49,7 @@ class FFmpegPostProcessor(PostProcessor):          for path in input_paths:              files_cmd.extend(['-i', encodeFilename(path, True)])          cmd = ([self._get_executable(), '-y'] + files_cmd -               + opts + +               + [encodeArgument(o) for o in opts] +                 [encodeFilename(self._ffmpeg_filename_argument(out_path), True)])          if self._downloader.params.get('verbose', False): diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py index aa9347f43..b97e62ae9 100644 --- a/youtube_dl/utils.py +++ b/youtube_dl/utils.py @@ -540,6 +540,16 @@ def encodeFilename(s, for_subprocess=False):          encoding = 'utf-8'      return s.encode(encoding, 'ignore') + +def encodeArgument(s): +    if not isinstance(s, compat_str): +        # Legacy code that uses byte strings +        # Uncomment the following line after fixing all post processors +        #assert False, 'Internal error: %r should be of type %r, is %r' % (s, compat_str, type(s)) +        s = s.decode('ascii') +    return encodeFilename(s, True) + +  def decodeOption(optval):      if optval is None:          return optval | 
