Unverified Commit 719e1203 authored by Prabhat Roy's avatar Prabhat Roy Committed by GitHub
Browse files

Ignore platforms other than Linux for ffmpeg (#4410)

* Fix ffmpeg version

* Ignore platforms other than Linux for ffmpeg

* Removed unnecessary changes
parent 921b6bef
...@@ -351,6 +351,8 @@ def get_extensions(): ...@@ -351,6 +351,8 @@ def get_extensions():
ffmpeg_exe = distutils.spawn.find_executable('ffmpeg') ffmpeg_exe = distutils.spawn.find_executable('ffmpeg')
has_ffmpeg = ffmpeg_exe is not None has_ffmpeg = ffmpeg_exe is not None
if sys.platform != 'linux':
has_ffmpeg = False
if has_ffmpeg: if has_ffmpeg:
try: try:
# this splits on both dots and spaces as the output format differs across versions / platforms # this splits on both dots and spaces as the output format differs across versions / platforms
...@@ -360,7 +362,7 @@ def get_extensions(): ...@@ -360,7 +362,7 @@ def get_extensions():
if StrictVersion(ffmpeg_version) >= StrictVersion('4.3'): if StrictVersion(ffmpeg_version) >= StrictVersion('4.3'):
print(f'ffmpeg {ffmpeg_version} not supported yet, please use ffmpeg 4.2.') print(f'ffmpeg {ffmpeg_version} not supported yet, please use ffmpeg 4.2.')
has_ffmpeg = False has_ffmpeg = False
except (IndexError, ValueError): except (subprocess.CalledProcessError, IndexError, ValueError):
print('Error fetching ffmpeg version, ignoring ffmpeg.') print('Error fetching ffmpeg version, ignoring ffmpeg.')
has_ffmpeg = False has_ffmpeg = False
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment