Merge pull request #229 from Dliwk/add-rsync-to-checkenv

Add rsync to checkenv
This commit is contained in:
Eric Froemling 2021-01-14 23:08:19 -08:00 committed by GitHub
commit 18c501ee22
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -500,6 +500,8 @@ def _vstr(nums: Sequence[int]) -> str:
def checkenv() -> None:
"""Check for tools necessary to build and run the app."""
# pylint: disable=too-many-branches
from efrotools import PYTHON_BIN
print(f'{Clr.BLD}Checking environment...{Clr.RST}', flush=True)
@ -509,6 +511,12 @@ def checkenv() -> None:
raise CleanError('curl is required; please install it via apt,'
' brew, etc.')
# Make sure they've got rsync.
if subprocess.run(['which', 'rsync'], check=False,
capture_output=True).returncode != 0:
raise CleanError('rsync is required; please install it via apt,'
' brew, etc.')
# Make sure they've got our target python version.
if subprocess.run(['which', PYTHON_BIN], check=False,
capture_output=True).returncode != 0: