xenofem/python-zulip-api
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

provision: Give a clear error message when virtualenv is missing.

Browse source 
Without this tweak, running the script results in a vague "No such file or
directory" error if the virtualenv package is not installed.
This commit is contained in:
Alena Volkova 2017-09-26 20:35:21 -04:00 committed by Tim Abbott
parent 449d5faa59
commit 21bc4778b0
1 changed files with 11 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
tools/provision
View file

@ -24,10 +24,18 @@ the Python version this command is executed with."""
venv_dir = os.path.join(base_dir, venv_name) venv_dir = os.path.join(base_dir, venv_name)
if not os.path.isdir(venv_dir): if not os.path.isdir(venv_dir):
if subprocess.call(['virtualenv', '-p', python_interpreter, venv_dir]): try:
return_code = subprocess.call(['virtualenv', '-p', python_interpreter, venv_dir])
except OSError:
if subprocess.call(['which', 'virtualenv']):
print("{red}Please install the virtualenv package and try again.{end_format}"
.format(red='\033[91m', end_format='\033[0m'))
sys.exit(1)
raise
else:
if return_code:
raise OSError("The command `virtualenv -p {} {}` failed. Virtualenv not created!" raise OSError("The command `virtualenv -p {} {}` failed. Virtualenv not created!"
.format(python_interpreter, venv_dir)) .format(python_interpreter, venv_dir))
else:
print("New virtualenv created.") print("New virtualenv created.")
else: else:
print("Virtualenv already exists.") print("Virtualenv already exists.")