1
0
mirror of https://github.com/yt-dlp/yt-dlp.git synced 2025-12-24 17:08:53 +00:00

Fixed embedding filepath issue for subs and infojson

This commit is contained in:
Kieran Eglin
2024-04-26 16:16:56 -07:00
parent 3046c17822
commit 6c8ede8188
3 changed files with 7 additions and 2 deletions

View File

@@ -226,6 +226,6 @@ class EmbedThumbnailPP(FFmpegPostProcessor):
info=info)
if not self._already_have_thumbnail:
del info['thumbnails'][idx]['filepath']
info['thumbnails'][idx].pop('filepath', None)
return [], info

View File

@@ -662,6 +662,10 @@ class FFmpegEmbedSubtitlePP(FFmpegPostProcessor):
self.run_ffmpeg_multiple_files(input_files, temp_filename, opts)
os.replace(temp_filename, filename)
if not self._already_have_subtitle:
for _, subtitle in subtitles.items():
subtitle.pop('filepath', None)
files_to_delete = [] if self._already_have_subtitle else sub_filenames
return files_to_delete, info
@@ -698,6 +702,7 @@ class FFmpegMetadataPP(FFmpegPostProcessor):
infojson_filename = info.get('infojson_filename')
options.extend(self._get_infojson_opts(info, infojson_filename))
if not infojson_filename:
info.pop('infojson_filename', None)
files_to_delete.append(info.get('infojson_filename'))
elif self._add_infojson is True:
self.to_screen('The info-json can only be attached to mkv/mka files')