fix: make install uses uv tool install without requiring a venv
uv pip install needs a virtual environment. Use uv tool install for the global CLI path and make sync for editable .venv development installs.
This commit is contained in:
parent
fcea33924b
commit
d72f64d7a0
2 changed files with 12 additions and 6 deletions
15
Makefile
15
Makefile
|
|
@ -3,13 +3,14 @@ ROOT := $(dir $(abspath $(lastword $(MAKEFILE_LIST))))
|
|||
|
||||
.DEFAULT_GOAL := help
|
||||
|
||||
.PHONY: help install install-editable check test report-7d report
|
||||
.PHONY: help sync install install-editable check test report-7d report
|
||||
|
||||
help:
|
||||
@echo "phase-memory make targets"
|
||||
@echo ""
|
||||
@echo " make install # pip install this package (user/site)"
|
||||
@echo " make install-editable # editable install for local development"
|
||||
@echo " make install # install phase-memory CLI (uv tool, no venv)"
|
||||
@echo " make sync # create .venv and editable install (dev)"
|
||||
@echo " make install-editable # alias for make sync"
|
||||
@echo " make check # verify import and print version"
|
||||
@echo " make test # run pytest"
|
||||
@echo " make report-7d # federated activity report (last 7 days)"
|
||||
|
|
@ -17,10 +18,14 @@ help:
|
|||
@echo ""
|
||||
@echo "For ops-warden integration, prefer: cd ../ops-warden && make install-all"
|
||||
|
||||
install-editable:
|
||||
sync:
|
||||
uv venv --allow-existing
|
||||
uv pip install -e "$(ROOT)"
|
||||
|
||||
install: install-editable
|
||||
install-editable: sync
|
||||
|
||||
install:
|
||||
uv tool install . --force
|
||||
|
||||
check:
|
||||
@PYTHONPATH="$(ROOT)src" python3 -c "import phase_memory; print('phase-memory', phase_memory.__version__, 'ok')"
|
||||
|
|
|
|||
|
|
@ -36,7 +36,8 @@ make test
|
|||
Install for local development or ops-warden integration:
|
||||
|
||||
```bash
|
||||
make install-editable
|
||||
make install # global CLI: phase-memory, phase-memory-service
|
||||
make sync # dev: editable install into .venv
|
||||
# or from ops-warden: make install-all
|
||||
```
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue