Skip to content

Commit

Permalink
- [#367] Fix setting the default value for SYSTEM_PYTHON env var in…
Browse files Browse the repository at this point in the history
… the `Makefile`.

- [#367] Fix `build` target in the `Makefile` to first activate the virtual environment and then install the requirements.
  • Loading branch information
eoyilmaz committed Oct 10, 2024
1 parent ab62a51 commit b71dcd1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ SETUP_PY_FLAGS = --use-distutils
VERSION := $(shell cat VERSION_BASE)
VERSION_FILE=$(CURDIR)/VERSION_BASE
VIRTUALENV_DIR:=.venv
SYSTEM_PYTHON:=python3
SYSTEM_PYTHON?=python3

all: build FORCE

Expand All @@ -19,10 +19,10 @@ venv:
$(SYSTEM_PYTHON) -m venv $(VIRTUALENV_DIR); \
source ./$(VIRTUALENV_DIR)/bin/activate; \

build: venv FORCE
build:
source ./$(VIRTUALENV_DIR)/bin/activate; \
pip install -r requirements.txt; \
pip install -r requirements-dev.txt;
source ./$(VIRTUALENV_DIR)/bin/activate; \
$(SYSTEM_PYTHON) -m build;

install:
Expand Down

0 comments on commit b71dcd1

Please sign in to comment.