Skip to content

Commit b8ad32a

Browse files
dirkfGitHub Actions
authored and
GitHub Actions
committed
[postprocessor/ffmpeg] Fix finding ffprobe (bug in 21792b8)
Fixes ytdl-org/youtube-dl@21792b8#commitcomment-140705274, thx: vonProteus
1 parent 4b84e03 commit b8ad32a

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

youtube_dl/postprocessor/ffmpeg.py

+11-5
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,11 @@ def get_versions(downloader=None):
7474
return FFmpegPostProcessor(downloader)._versions
7575

7676
def _determine_executables(self):
77-
programs = ['avprobe', 'avconv', 'ffmpeg', 'ffprobe']
77+
# ordered to match prefer_ffmpeg!
78+
convs = ['ffmpeg', 'avconv']
79+
probes = ['ffprobe', 'avprobe']
7880
prefer_ffmpeg = True
81+
programs = convs + probes
7982

8083
def get_ffmpeg_version(path):
8184
ver = get_exe_version(path, args=['-version'])
@@ -127,10 +130,13 @@ def get_ffmpeg_version(path):
127130
(p, get_ffmpeg_version(self._paths[p])) for p in programs)
128131
if x[1] is not None)
129132

130-
for p in ('ffmpeg', 'avconv')[::-1 if prefer_ffmpeg is False else 1]:
131-
if self._versions.get(p):
132-
self.basename = self.probe_basename = p
133-
break
133+
basenames = [None, None]
134+
for i, progs in enumerate((convs, probes)):
135+
for p in progs[::-1 if prefer_ffmpeg is False else 1]:
136+
if self._versions.get(p):
137+
basenames[i] = p
138+
break
139+
self.basename, self.probe_basename = basenames
134140

135141
@property
136142
def available(self):

0 commit comments

Comments
 (0)