[ExtractAudio] Handle outtmpl without ext (#6005)

Authored by: carusocr
Closes #5968
pull/5822/head^2
Chris Caruso 2 years ago committed by GitHub
parent 5e1a54f63e
commit f737fb16d8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -318,10 +318,6 @@ def validate_options(opts):
if outtmpl_default == '': if outtmpl_default == '':
opts.skip_download = None opts.skip_download = None
del opts.outtmpl['default'] del opts.outtmpl['default']
if outtmpl_default and not os.path.splitext(outtmpl_default)[1] and opts.extractaudio:
raise ValueError(
'Cannot download a video and extract audio into the same file! '
f'Use "{outtmpl_default}.%(ext)s" instead of "{outtmpl_default}" as the output template')
def parse_chapters(name, value): def parse_chapters(name, value):
chapters, ranges = [], [] chapters, ranges = [], []

@ -508,8 +508,7 @@ class FFmpegExtractAudioPP(FFmpegPostProcessor):
if acodec != 'copy': if acodec != 'copy':
more_opts = self._quality_args(acodec) more_opts = self._quality_args(acodec)
# not os.path.splitext, since the latter does not work on unicode in all setups temp_path = new_path = replace_extension(path, extension, information['ext'])
temp_path = new_path = f'{path.rpartition(".")[0]}.{extension}'
if new_path == path: if new_path == path:
if acodec == 'copy': if acodec == 'copy':

Loading…
Cancel
Save