techbloc/techrec
1
0
Fork 0
Code Issues 10 Pull requests Releases Activity

Closes #17: warn if ffmpeg binary is not found

Browse source
This commit is contained in:
boyska 2014-04-14 18:51:04 +02:00
parent 65aad94f24
commit c135cbf573
1 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
server/cli.py
View file

@ -34,6 +34,19 @@ def pre_check_user():
if os.geteuid() == 0: if os.geteuid() == 0:
yield "You're running as root; this is dangerous" yield "You're running as root; this is dangerous"
def pre_check_ffmpeg():
path = get_config()['FFMPEG_PATH']
if not path.startswith('/'):
yield "FFMPEG_PATH is not absolute: %s" % path
from subprocess import check_output
try:
check_output([path, '-version'])
except OSError:
yield "FFMPEG not found as " + path
else:
if not os.path.exists(path):
yield "FFMPEG not found in " + path
class DateTimeAction(Action): class DateTimeAction(Action):
def __call__(self, parser, namespace, values, option_string=None): def __call__(self, parser, namespace, values, option_string=None):
@ -45,7 +58,7 @@ class DateTimeAction(Action):
def common_pre(): def common_pre():
prechecks = [pre_check_user, pre_check_permissions] prechecks = [pre_check_user, pre_check_permissions, pre_check_ffmpeg]
configs = ['default_config.py'] configs = ['default_config.py']
if 'TECHREC_CONFIG' in os.environ: if 'TECHREC_CONFIG' in os.environ:
for conf in os.environ['TECHREC_CONFIG'].split(':'): for conf in os.environ['TECHREC_CONFIG'].split(':'):