w/youtube-dl
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Revert "[ffmpeg] Fix embedding subtitles (#9063)"

Browse source 
This reverts commit ccff2c404d.

Fixes #10081.

The new approach breaks embedding subtitles into video-only or
audio-only files. FFMpeg provides a trick: add '?' after the argument of
'-map' so that a missing stream is ignored. For example:

opts = [
    '-map', '0:v?',
    '-c:v', 'copy',
    '-map', '0:a?',
    '-c:a', 'copy',
    # other options...
]

Unfortunately, such a format is not implemented in avconv, either.
I guess adding '-ignore_unknown' if self.basename == 'ffmpeg' is the
best solution. However, the example mentioned in #9063 no longer serves
problematic files, so I can't test it. I'll reopen #9063 and wait for
another example so that I can test '-ignore_unknown'.
This commit is contained in:
Yen Chi Hsuan 2016-07-15 19:53:10 +08:00
parent 224db034ab
commit 23495d6a39
No known key found for this signature in database
GPG key ID: 3FDDD575826C5C30
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
youtube_dl/postprocessor/ffmpeg.py
View file

@ -363,10 +363,8 @@ class FFmpegEmbedSubtitlePP(FFmpegPostProcessor):
input_files = [filename] + sub_filenames input_files = [filename] + sub_filenames
opts = [ opts = [
'-map', '0:v', '-map', '0',
'-c:v', 'copy', '-c', 'copy',
'-map', '0:a',
'-c:a', 'copy',
# Don't copy the existing subtitles, we may be running the # Don't copy the existing subtitles, we may be running the
# postprocessor a second time # postprocessor a second time
'-map', '-0:s', '-map', '-0:s',