From 0163ada401e84ba0aa116c4e011538f8d95855fa Mon Sep 17 00:00:00 2001 From: Oz N Tiram Date: Thu, 12 May 2022 19:55:45 +0200 Subject: [PATCH] Makefile fixes: don't force recrating of virtualenv Everytime that we run the tests we create a temporary virtualenv. This is nice for idempotency, but really a killer for SSDs on developers machines. I would recommend also to document the ramdisks target, since running the tests in the ramdisk should be faster. --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index b78b4c0aad..8c4ed7d2f4 100644 --- a/Makefile +++ b/Makefile @@ -51,11 +51,11 @@ ramdisk-virtualenv: ramdisk .PHONY: virtualenv virtualenv: - [ ! -e $(venv_dir) ] && rm -rf $(venv_file) && python -m venv $(venv_dir) + [ ! -e $(venv_dir) ] && rm -rvf $(venv_file) && python -m venv $(venv_dir) @echo $(venv_dir) >> $(venv_file) .PHONY: test-install -test-install: virtualenv +test-install: . $(get_venv_path)/bin/activate && \ python -m pip install --upgrade pip -e .[tests,dev] && \ pipenv install --dev @@ -67,7 +67,7 @@ submodules: .PHONY: tests tests: parallel ?= -n auto -tests: virtualenv submodules test-install +tests: submodules test-install . $(get_venv_path)/bin/activate && \ pipenv run pytest -ra $(parallel) -vvv --full-trace --tb=long