_ Git - youtube-dl/blob - youtube_dl/postprocessor/ffmpeg.py

   1 from __future__ import unicode_literals
   2
   3 import io
   4 import os
   5 import subprocess
   6 import time
   7 import re
   8
   9
  10 from .common import AudioConversionError, PostProcessor
  11
  12 from ..utils import (
  13     encodeArgument,
  14     encodeFilename,
  15     get_exe_version,
  16     is_outdated_version,
  17     PostProcessingError,
  18     prepend_extension,
  19     shell_quote,
  20     subtitles_filename,
  21     dfxp2srt,
  22     ISO639Utils,
  23     replace_extension,
  24 )
  25
  26
  27 EXT_TO_OUT_FORMATS = {
  28     'aac': 'adts',
  29     'flac': 'flac',
  30     'm4a': 'ipod',
  31     'mka': 'matroska',
  32     'mkv': 'matroska',
  33     'mpg': 'mpeg',
  34     'ogv': 'ogg',
  35     'ts': 'mpegts',
  36     'wma': 'asf',
  37     'wmv': 'asf',
  38 }
  39 ACODECS = {
  40     'mp3': 'libmp3lame',
  41     'aac': 'aac',
  42     'flac': 'flac',
  43     'm4a': 'aac',
  44     'opus': 'libopus',
  45     'vorbis': 'libvorbis',
  46     'wav': None,
  47 }
  48
  49
  50 class FFmpegPostProcessorError(PostProcessingError):
  51     pass
  52
  53
  54 class FFmpegPostProcessor(PostProcessor):
  55     def __init__(self, downloader=None):
  56         PostProcessor.__init__(self, downloader)
  57         self._determine_executables()
  58
  59     def check_version(self):
  60         if not self.available:
  61             raise FFmpegPostProcessorError('ffmpeg or avconv not found. Please install one.')
  62
  63         required_version = '10-0' if self.basename == 'avconv' else '1.0'
  64         if is_outdated_version(
  65                 self._versions[self.basename], required_version):
  66             warning = 'Your copy of %s is outdated, update %s to version %s or newer if you encounter any errors.' % (
  67                 self.basename, self.basename, required_version)
  68             if self._downloader:
  69                 self._downloader.report_warning(warning)
  70
  71     @staticmethod
  72     def get_versions(downloader=None):
  73         return FFmpegPostProcessor(downloader)._versions
  74
  75     def _determine_executables(self):
  76         programs = ['avprobe', 'avconv', 'ffmpeg', 'ffprobe']
  77         prefer_ffmpeg = True
  78
  79         def get_ffmpeg_version(path):
  80             ver = get_exe_version(path, args=['-version'])
  81             if ver:
  82                 regexs = [
  83                     r'(?:\d+:)?([0-9.]+)-[0-9]+ubuntu[0-9.]+$',  # Ubuntu, see [1]
  84                     r'n([0-9.]+)$',  # Arch Linux
  85                     # 1. http://www.ducea.com/2006/06/17/ubuntu-package-version-naming-explanation/
  86                 ]
  87                 for regex in regexs:
  88                     mobj = re.match(regex, ver)
  89                     if mobj:
  90                         ver = mobj.group(1)
  91             return ver
  92
  93         self.basename = None
  94         self.probe_basename = None
  95
  96         self._paths = None
  97         self._versions = None
  98         if self._downloader:
  99             prefer_ffmpeg = self._downloader.params.get('prefer_ffmpeg', True)
 100             location = self._downloader.params.get('ffmpeg_location')
 101             if location is not None:
 102                 if not os.path.exists(location):
 103                     self._downloader.report_warning(
 104                         'ffmpeg-location %s does not exist! '
 105                         'Continuing without avconv/ffmpeg.' % (location))
 106                     self._versions = {}
 107                     return
 108                 elif not os.path.isdir(location):
 109                     basename = os.path.splitext(os.path.basename(location))[0]
 110                     if basename not in programs:
 111                         self._downloader.report_warning(
 112                             'Cannot identify executable %s, its basename should be one of %s. '
 113                             'Continuing without avconv/ffmpeg.' %
 114                             (location, ', '.join(programs)))
 115                         self._versions = {}
 116                         return None
 117                     location = os.path.dirname(os.path.abspath(location))
 118                     if basename in ('ffmpeg', 'ffprobe'):
 119                         prefer_ffmpeg = True
 120
 121                 self._paths = dict(
 122                     (p, os.path.join(location, p)) for p in programs)
 123                 self._versions = dict(
 124                     (p, get_ffmpeg_version(self._paths[p])) for p in programs)
 125         if self._versions is None:
 126             self._versions = dict(
 127                 (p, get_ffmpeg_version(p)) for p in programs)
 128             self._paths = dict((p, p) for p in programs)
 129
 130         if prefer_ffmpeg is False:
 131             prefs = ('avconv', 'ffmpeg')
 132         else:
 133             prefs = ('ffmpeg', 'avconv')
 134         for p in prefs:
 135             if self._versions[p]:
 136                 self.basename = p
 137                 break
 138
 139         if prefer_ffmpeg is False:
 140             prefs = ('avprobe', 'ffprobe')
 141         else:
 142             prefs = ('ffprobe', 'avprobe')
 143         for p in prefs:
 144             if self._versions[p]:
 145                 self.probe_basename = p
 146                 break
 147
 148     @property
 149     def available(self):
 150         return self.basename is not None
 151
 152     @property
 153     def executable(self):
 154         return self._paths[self.basename]
 155
 156     @property
 157     def probe_available(self):
 158         return self.probe_basename is not None
 159
 160     @property
 161     def probe_executable(self):
 162         return self._paths[self.probe_basename]
 163
 164     def get_audio_codec(self, path):
 165         if not self.probe_available and not self.available:
 166             raise PostProcessingError('ffprobe/avprobe and ffmpeg/avconv not found. Please install one.')
 167         try:
 168             if self.probe_available:
 169                 cmd = [
 170                     encodeFilename(self.probe_executable, True),
 171                     encodeArgument('-show_streams')]
 172             else:
 173                 cmd = [
 174                     encodeFilename(self.executable, True),
 175                     encodeArgument('-i')]
 176             cmd.append(encodeFilename(self._ffmpeg_filename_argument(path), True))
 177             if self._downloader.params.get('verbose', False):
 178                 self._downloader.to_screen(
 179                     '[debug] %s command line: %s' % (self.basename, shell_quote(cmd)))
 180             handle = subprocess.Popen(
 181                 cmd, stderr=subprocess.PIPE,
 182                 stdout=subprocess.PIPE, stdin=subprocess.PIPE)
 183             stdout_data, stderr_data = handle.communicate()
 184             expected_ret = 0 if self.probe_available else 1
 185             if handle.wait() != expected_ret:
 186                 return None
 187         except (IOError, OSError):
 188             return None
 189         output = (stdout_data if self.probe_available else stderr_data).decode('ascii', 'ignore')
 190         if self.probe_available:
 191             audio_codec = None
 192             for line in output.split('\n'):
 193                 if line.startswith('codec_name='):
 194                     audio_codec = line.split('=')[1].strip()
 195                 elif line.strip() == 'codec_type=audio' and audio_codec is not None:
 196                     return audio_codec
 197         else:
 198             # Stream #FILE_INDEX:STREAM_INDEX[STREAM_ID](LANGUAGE): CODEC_TYPE: CODEC_NAME
 199             mobj = re.search(
 200                 r'Stream\s*#\d+:\d+(?:\[0x[0-9a-f]+\])?(?:\([a-z]{3}\))?:\s*Audio:\s*([0-9a-z]+)',
 201                 output)
 202             if mobj:
 203                 return mobj.group(1)
 204         return None
 205
 206     def run_ffmpeg_multiple_files(self, input_paths, out_path, opts):
 207         self.check_version()
 208
 209         oldest_mtime = min(
 210             os.stat(encodeFilename(path)).st_mtime for path in input_paths)
 211
 212         opts += self._configuration_args()
 213
 214         files_cmd = []
 215         for path in input_paths:
 216             files_cmd.extend([
 217                 encodeArgument('-i'),
 218                 encodeFilename(self._ffmpeg_filename_argument(path), True)
 219             ])
 220         cmd = ([encodeFilename(self.executable, True), encodeArgument('-y')] +
 221                files_cmd +
 222                [encodeArgument(o) for o in opts] +
 223                [encodeFilename(self._ffmpeg_filename_argument(out_path), True)])
 224
 225         if self._downloader.params.get('verbose', False):
 226             self._downloader.to_screen('[debug] ffmpeg command line: %s' % shell_quote(cmd))
 227         p = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, stdin=subprocess.PIPE)
 228         stdout, stderr = p.communicate()
 229         if p.returncode != 0:
 230             stderr = stderr.decode('utf-8', 'replace')
 231             msg = stderr.strip().split('\n')[-1]
 232             raise FFmpegPostProcessorError(msg)
 233         self.try_utime(out_path, oldest_mtime, oldest_mtime)
 234
 235     def run_ffmpeg(self, path, out_path, opts):
 236         self.run_ffmpeg_multiple_files([path], out_path, opts)
 237
 238     def _ffmpeg_filename_argument(self, fn):
 239         # Always use 'file:' because the filename may contain ':' (ffmpeg
 240         # interprets that as a protocol) or can start with '-' (-- is broken in
 241         # ffmpeg, see https://ffmpeg.org/trac/ffmpeg/ticket/2127 for details)
 242         # Also leave '-' intact in order not to break streaming to stdout.
 243         return 'file:' + fn if fn != '-' else fn
 244
 245
 246 class FFmpegExtractAudioPP(FFmpegPostProcessor):
 247     def __init__(self, downloader=None, preferredcodec=None, preferredquality=None, nopostoverwrites=False):
 248         FFmpegPostProcessor.__init__(self, downloader)
 249         if preferredcodec is None:
 250             preferredcodec = 'best'
 251         self._preferredcodec = preferredcodec
 252         self._preferredquality = preferredquality
 253         self._nopostoverwrites = nopostoverwrites
 254
 255     def run_ffmpeg(self, path, out_path, codec, more_opts):
 256         if codec is None:
 257             acodec_opts = []
 258         else:
 259             acodec_opts = ['-acodec', codec]
 260         opts = ['-vn'] + acodec_opts + more_opts
 261         try:
 262             FFmpegPostProcessor.run_ffmpeg(self, path, out_path, opts)
 263         except FFmpegPostProcessorError as err:
 264             raise AudioConversionError(err.msg)
 265
 266     def run(self, information):
 267         path = information['filepath']
 268
 269         filecodec = self.get_audio_codec(path)
 270         if filecodec is None:
 271             raise PostProcessingError('WARNING: unable to obtain file audio codec with ffprobe')
 272
 273         more_opts = []
 274         if self._preferredcodec == 'best' or self._preferredcodec == filecodec or (self._preferredcodec == 'm4a' and filecodec == 'aac'):
 275             if filecodec == 'aac' and self._preferredcodec in ['m4a', 'best']:
 276                 # Lossless, but in another container
 277                 acodec = 'copy'
 278                 extension = 'm4a'
 279                 more_opts = ['-bsf:a', 'aac_adtstoasc']
 280             elif filecodec in ['aac', 'flac', 'mp3', 'vorbis', 'opus']:
 281                 # Lossless if possible
 282                 acodec = 'copy'
 283                 extension = filecodec
 284                 if filecodec == 'aac':
 285                     more_opts = ['-f', 'adts']
 286                 if filecodec == 'vorbis':
 287                     extension = 'ogg'
 288             else:
 289                 # MP3 otherwise.
 290                 acodec = 'libmp3lame'
 291                 extension = 'mp3'
 292                 more_opts = []
 293                 if self._preferredquality is not None:
 294                     if int(self._preferredquality) < 10:
 295                         more_opts += ['-q:a', self._preferredquality]
 296                     else:
 297                         more_opts += ['-b:a', self._preferredquality + 'k']
 298         else:
 299             # We convert the audio (lossy if codec is lossy)
 300             acodec = ACODECS[self._preferredcodec]
 301             extension = self._preferredcodec
 302             more_opts = []
 303             if self._preferredquality is not None:
 304                 # The opus codec doesn't support the -aq option
 305                 if int(self._preferredquality) < 10 and extension != 'opus':
 306                     more_opts += ['-q:a', self._preferredquality]
 307                 else:
 308                     more_opts += ['-b:a', self._preferredquality + 'k']
 309             if self._preferredcodec == 'aac':
 310                 more_opts += ['-f', 'adts']
 311             if self._preferredcodec == 'm4a':
 312                 more_opts += ['-bsf:a', 'aac_adtstoasc']
 313             if self._preferredcodec == 'vorbis':
 314                 extension = 'ogg'
 315             if self._preferredcodec == 'wav':
 316                 extension = 'wav'
 317                 more_opts += ['-f', 'wav']
 318
 319         prefix, sep, ext = path.rpartition('.')  # not os.path.splitext, since the latter does not work on unicode in all setups
 320         new_path = prefix + sep + extension
 321
 322         information['filepath'] = new_path
 323         information['ext'] = extension
 324
 325         # If we download foo.mp3 and convert it to... foo.mp3, then don't delete foo.mp3, silly.
 326         if (new_path == path or
 327                 (self._nopostoverwrites and os.path.exists(encodeFilename(new_path)))):
 328             self._downloader.to_screen('[ffmpeg] Post-process file %s exists, skipping' % new_path)
 329             return [], information
 330
 331         try:
 332             self._downloader.to_screen('[ffmpeg] Destination: ' + new_path)
 333             self.run_ffmpeg(path, new_path, acodec, more_opts)
 334         except AudioConversionError as e:
 335             raise PostProcessingError(
 336                 'audio conversion failed: ' + e.msg)
 337         except Exception:
 338             raise PostProcessingError('error running ' + self.basename)
 339
 340         # Try to update the date time for extracted audio file.
 341         if information.get('filetime') is not None:
 342             self.try_utime(
 343                 new_path, time.time(), information['filetime'],
 344                 errnote='Cannot update utime of audio file')
 345
 346         return [path], information
 347
 348
 349 class FFmpegVideoConvertorPP(FFmpegPostProcessor):
 350     def __init__(self, downloader=None, preferedformat=None):
 351         super(FFmpegVideoConvertorPP, self).__init__(downloader)
 352         self._preferedformat = preferedformat
 353
 354     def run(self, information):
 355         path = information['filepath']
 356         if information['ext'] == self._preferedformat:
 357             self._downloader.to_screen('[ffmpeg] Not converting video file %s - already is in target format %s' % (path, self._preferedformat))
 358             return [], information
 359         options = []
 360         if self._preferedformat == 'avi':
 361             options.extend(['-c:v', 'libxvid', '-vtag', 'XVID'])
 362         prefix, sep, ext = path.rpartition('.')
 363         outpath = prefix + sep + self._preferedformat
 364         self._downloader.to_screen('[' + 'ffmpeg' + '] Converting video from %s to %s, Destination: ' % (information['ext'], self._preferedformat) + outpath)
 365         self.run_ffmpeg(path, outpath, options)
 366         information['filepath'] = outpath
 367         information['format'] = self._preferedformat
 368         information['ext'] = self._preferedformat
 369         return [path], information
 370
 371
 372 class FFmpegEmbedSubtitlePP(FFmpegPostProcessor):
 373     def run(self, information):
 374         if information['ext'] not in ('mp4', 'webm', 'mkv'):
 375             self._downloader.to_screen('[ffmpeg] Subtitles can only be embedded in mp4, webm or mkv files')
 376             return [], information
 377         subtitles = information.get('requested_subtitles')
 378         if not subtitles:
 379             self._downloader.to_screen('[ffmpeg] There aren\'t any subtitles to embed')
 380             return [], information
 381
 382         filename = information['filepath']
 383
 384         ext = information['ext']
 385         sub_langs = []
 386         sub_filenames = []
 387         webm_vtt_warn = False
 388
 389         for lang, sub_info in subtitles.items():
 390             sub_ext = sub_info['ext']
 391             if ext != 'webm' or ext == 'webm' and sub_ext == 'vtt':
 392                 sub_langs.append(lang)
 393                 sub_filenames.append(subtitles_filename(filename, lang, sub_ext))
 394             else:
 395                 if not webm_vtt_warn and ext == 'webm' and sub_ext != 'vtt':
 396                     webm_vtt_warn = True
 397                     self._downloader.to_screen('[ffmpeg] Only WebVTT subtitles can be embedded in webm files')
 398
 399         if not sub_langs:
 400             return [], information
 401
 402         input_files = [filename] + sub_filenames
 403
 404         opts = [
 405             '-map', '0',
 406             '-c', 'copy',
 407             # Don't copy the existing subtitles, we may be running the
 408             # postprocessor a second time
 409             '-map', '-0:s',
 410         ]
 411         if information['ext'] == 'mp4':
 412             opts += ['-c:s', 'mov_text']
 413         for (i, lang) in enumerate(sub_langs):
 414             opts.extend(['-map', '%d:0' % (i + 1)])
 415             lang_code = ISO639Utils.short2long(lang) or lang
 416             opts.extend(['-metadata:s:s:%d' % i, 'language=%s' % lang_code])
 417
 418         temp_filename = prepend_extension(filename, 'temp')
 419         self._downloader.to_screen('[ffmpeg] Embedding subtitles in \'%s\'' % filename)
 420         self.run_ffmpeg_multiple_files(input_files, temp_filename, opts)
 421         os.remove(encodeFilename(filename))
 422         os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 423
 424         return sub_filenames, information
 425
 426
 427 class FFmpegMetadataPP(FFmpegPostProcessor):
 428     def run(self, info):
 429         metadata = {}
 430
 431         def add(meta_list, info_list=None):
 432             if not info_list:
 433                 info_list = meta_list
 434             if not isinstance(meta_list, (list, tuple)):
 435                 meta_list = (meta_list,)
 436             if not isinstance(info_list, (list, tuple)):
 437                 info_list = (info_list,)
 438             for info_f in info_list:
 439                 if info.get(info_f) is not None:
 440                     for meta_f in meta_list:
 441                         metadata[meta_f] = info[info_f]
 442                     break
 443
 444         add('title', ('track', 'title'))
 445         add('date', 'upload_date')
 446         add(('description', 'comment'), 'description')
 447         add('purl', 'webpage_url')
 448         add('track', 'track_number')
 449         add('artist', ('artist', 'creator', 'uploader', 'uploader_id'))
 450         add('genre')
 451         add('album')
 452         add('album_artist')
 453         add('disc', 'disc_number')
 454
 455         if not metadata:
 456             self._downloader.to_screen('[ffmpeg] There isn\'t any metadata to add')
 457             return [], info
 458
 459         filename = info['filepath']
 460         temp_filename = prepend_extension(filename, 'temp')
 461         in_filenames = [filename]
 462         options = []
 463
 464         if info['ext'] == 'm4a':
 465             options.extend(['-vn', '-acodec', 'copy'])
 466         else:
 467             options.extend(['-c', 'copy'])
 468
 469         for (name, value) in metadata.items():
 470             options.extend(['-metadata', '%s=%s' % (name, value)])
 471
 472         chapters = info.get('chapters', [])
 473         if chapters:
 474             metadata_filename = replace_extension(filename, 'meta')
 475             with io.open(metadata_filename, 'wt', encoding='utf-8') as f:
 476                 def ffmpeg_escape(text):
 477                     return re.sub(r'(=|;|#|\\|\n)', r'\\\1', text)
 478
 479                 metadata_file_content = ';FFMETADATA1\n'
 480                 for chapter in chapters:
 481                     metadata_file_content += '[CHAPTER]\nTIMEBASE=1/1000\n'
 482                     metadata_file_content += 'START=%d\n' % (chapter['start_time'] * 1000)
 483                     metadata_file_content += 'END=%d\n' % (chapter['end_time'] * 1000)
 484                     chapter_title = chapter.get('title')
 485                     if chapter_title:
 486                         metadata_file_content += 'title=%s\n' % ffmpeg_escape(chapter_title)
 487                 f.write(metadata_file_content)
 488                 in_filenames.append(metadata_filename)
 489                 options.extend(['-map_metadata', '1'])
 490
 491         self._downloader.to_screen('[ffmpeg] Adding metadata to \'%s\'' % filename)
 492         self.run_ffmpeg_multiple_files(in_filenames, temp_filename, options)
 493         if chapters:
 494             os.remove(metadata_filename)
 495         os.remove(encodeFilename(filename))
 496         os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 497         return [], info
 498
 499
 500 class FFmpegMergerPP(FFmpegPostProcessor):
 501     def run(self, info):
 502         filename = info['filepath']
 503         temp_filename = prepend_extension(filename, 'temp')
 504         args = ['-c', 'copy', '-map', '0:v:0', '-map', '1:a:0']
 505         self._downloader.to_screen('[ffmpeg] Merging formats into "%s"' % filename)
 506         self.run_ffmpeg_multiple_files(info['__files_to_merge'], temp_filename, args)
 507         os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 508         return info['__files_to_merge'], info
 509
 510     def can_merge(self):
 511         # TODO: figure out merge-capable ffmpeg version
 512         if self.basename != 'avconv':
 513             return True
 514
 515         required_version = '10-0'
 516         if is_outdated_version(
 517                 self._versions[self.basename], required_version):
 518             warning = ('Your copy of %s is outdated and unable to properly mux separate video and audio files, '
 519                        'youtube-dl will download single file media. '
 520                        'Update %s to version %s or newer to fix this.') % (
 521                            self.basename, self.basename, required_version)
 522             if self._downloader:
 523                 self._downloader.report_warning(warning)
 524             return False
 525         return True
 526
 527
 528 class FFmpegFixupStretchedPP(FFmpegPostProcessor):
 529     def run(self, info):
 530         stretched_ratio = info.get('stretched_ratio')
 531         if stretched_ratio is None or stretched_ratio == 1:
 532             return [], info
 533
 534         filename = info['filepath']
 535         temp_filename = prepend_extension(filename, 'temp')
 536
 537         options = ['-c', 'copy', '-aspect', '%f' % stretched_ratio]
 538         self._downloader.to_screen('[ffmpeg] Fixing aspect ratio in "%s"' % filename)
 539         self.run_ffmpeg(filename, temp_filename, options)
 540
 541         os.remove(encodeFilename(filename))
 542         os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 543
 544         return [], info
 545
 546
 547 class FFmpegFixupM4aPP(FFmpegPostProcessor):
 548     def run(self, info):
 549         if info.get('container') != 'm4a_dash':
 550             return [], info
 551
 552         filename = info['filepath']
 553         temp_filename = prepend_extension(filename, 'temp')
 554
 555         options = ['-c', 'copy', '-f', 'mp4']
 556         self._downloader.to_screen('[ffmpeg] Correcting container in "%s"' % filename)
 557         self.run_ffmpeg(filename, temp_filename, options)
 558
 559         os.remove(encodeFilename(filename))
 560         os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 561
 562         return [], info
 563
 564
 565 class FFmpegFixupM3u8PP(FFmpegPostProcessor):
 566     def run(self, info):
 567         filename = info['filepath']
 568         if self.get_audio_codec(filename) == 'aac':
 569             temp_filename = prepend_extension(filename, 'temp')
 570
 571             options = ['-c', 'copy', '-f', 'mp4', '-bsf:a', 'aac_adtstoasc']
 572             self._downloader.to_screen('[ffmpeg] Fixing malformed AAC bitstream in "%s"' % filename)
 573             self.run_ffmpeg(filename, temp_filename, options)
 574
 575             os.remove(encodeFilename(filename))
 576             os.rename(encodeFilename(temp_filename), encodeFilename(filename))
 577         return [], info
 578
 579
 580 class FFmpegSubtitlesConvertorPP(FFmpegPostProcessor):
 581     def __init__(self, downloader=None, format=None):
 582         super(FFmpegSubtitlesConvertorPP, self).__init__(downloader)
 583         self.format = format
 584
 585     def run(self, info):
 586         subs = info.get('requested_subtitles')
 587         filename = info['filepath']
 588         new_ext = self.format
 589         new_format = new_ext
 590         if new_format == 'vtt':
 591             new_format = 'webvtt'
 592         if subs is None:
 593             self._downloader.to_screen('[ffmpeg] There aren\'t any subtitles to convert')
 594             return [], info
 595         self._downloader.to_screen('[ffmpeg] Converting subtitles')
 596         sub_filenames = []
 597         for lang, sub in subs.items():
 598             ext = sub['ext']
 599             if ext == new_ext:
 600                 self._downloader.to_screen(
 601                     '[ffmpeg] Subtitle file for %s is already in the requested format' % new_ext)
 602                 continue
 603             old_file = subtitles_filename(filename, lang, ext)
 604             sub_filenames.append(old_file)
 605             new_file = subtitles_filename(filename, lang, new_ext)
 606
 607             if ext in ('dfxp', 'ttml', 'tt'):
 608                 self._downloader.report_warning(
 609                     'You have requested to convert dfxp (TTML) subtitles into another format, '
 610                     'which results in style information loss')
 611
 612                 dfxp_file = old_file
 613                 srt_file = subtitles_filename(filename, lang, 'srt')
 614
 615                 with open(dfxp_file, 'rb') as f:
 616                     srt_data = dfxp2srt(f.read())
 617
 618                 with io.open(srt_file, 'wt', encoding='utf-8') as f:
 619                     f.write(srt_data)
 620                 old_file = srt_file
 621
 622                 subs[lang] = {
 623                     'ext': 'srt',
 624                     'data': srt_data
 625                 }
 626
 627                 if new_ext == 'srt':
 628                     continue
 629                 else:
 630                     sub_filenames.append(srt_file)
 631
 632             self.run_ffmpeg(old_file, new_file, ['-f', new_format])
 633
 634             with io.open(new_file, 'rt', encoding='utf-8') as f:
 635                 subs[lang] = {
 636                     'ext': new_ext,
 637                     'data': f.read(),
 638                 }
 639
 640         return sub_filenames, info