Merge e8bebe0b03d256371c38a983d7a6d2147d827cba into 4e714f9df1ed2cccd51df60d45ff5504abe827b7

[postprocessor/ffmpeg] Allow options for mp3
Why? Options like `preferredquality` are ignored when postprocessing mp3s. What? Remove `mp3` from the list of `filecodec` values to check for in the "lossless" condition unless `preferredquality` is unspecified.
2025-05-12 00:34:50 +09:00 · 2025-03-26 19:27:10 +05:45 · 2023-08-17 07:48:20 -07:00
1 changed files with 4 additions and 1 deletions
--- a/youtube_dl/postprocessor/ffmpeg.py
+++ b/youtube_dl/postprocessor/ffmpeg.py
@ -276,7 +276,10 @@ class FFmpegExtractAudioPP(FFmpegPostProcessor):
                acodec = 'copy'
                extension = 'm4a'
                more_opts = ['-bsf:a', 'aac_adtstoasc']
-            elif filecodec in ['aac', 'flac', 'mp3', 'vorbis', 'opus']:
+            elif (
+                filecodec in ['aac', 'flac', 'vorbis', 'opus'] or
+                (filecodec == 'mp3' and self._preferredquality is None)
+            ):
                # Lossless if possible
                acodec = 'copy'
                extension = filecodec
Author	SHA1	Message	Date
Alex Richards	1107557691	Merge e8bebe0b03d256371c38a983d7a6d2147d827cba into 4e714f9df1ed2cccd51df60d45ff5504abe827b7	2025-03-26 19:27:10 +05:45
Alex Richards	e8bebe0b03	[postprocessor/ffmpeg] Allow options for mp3 Why? Options like `preferredquality` are ignored when postprocessing mp3s. What? Remove `mp3` from the list of `filecodec` values to check for in the "lossless" condition unless `preferredquality` is unspecified.	2023-08-17 07:48:20 -07:00