Skip to content

Commit 1815d20

Browse files
author
Tyler Nienhouse
authored
Merge pull request #22 from Learnosity/build/fixup-build-venv-deps
[BUILD] Fix up venv dependencies in Makefile
2 parents 62195f7 + 688b525 commit 1815d20

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Makefile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ release:
2222
@./release.sh
2323
@echo '*** You can now use \`make dist-upload\` to publish the new version to PyPI'
2424

25-
freeze-deps:
25+
freeze-deps: venv
2626
$(call venv-activate); \
2727
$(PYTHON) -m pip freeze | grep -v learnosity_sdk > requirements.txt
2828

@@ -42,7 +42,7 @@ test-integration-dev: venv pip-tox
4242

4343
build-clean: real-clean
4444

45-
dist: distclean
45+
dist: distclean venv
4646
$(call venv-activate); \
4747
$(PYTHON) setup.py sdist; \
4848
$(PYTHON) setup.py bdist_wheel --universal
@@ -53,7 +53,7 @@ dist-check-version:
5353
ifeq ('$(shell echo $(GIT_TAG) | grep -qw "$(PKG_VER)")', '')
5454
$(error Version number $(PKG_VER) in learnosity_sdk/_version.py does not match git tag $(GIT_TAG))
5555
endif
56-
dist-upload-twine: pip-requirements-dev dist # This target doesn't do any safety check!
56+
dist-upload-twine: venv pip-requirements-dev dist # This target doesn't do any safety check!
5757
$(call venv-activate); \
5858
twine upload dist/*
5959

0 commit comments

Comments
 (0)