Repository navigation
Install vhlab-toolbox/library from GitHub archive URLs instead of git+ - #326
Conversation
uv and pip shell out to a system git for git+ URLs; archive URLs are fetched over plain HTTPS, so end users of the installer no longer need git for these two. The vhlab-toolbox-python URL must match vhlab-library-python's string exactly, or resolution fails with a conflicting-URL error. did, ndr, ndi-compress and ndi-ontology are left on git+ URLs for a later step. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UMLSmPQyLVBfBC52mCahmk
|
CI failures on 1.
The fix is VH-Lab/vhlab-library-python#12 (CI green there: lint plus tests on 3.10, 3.11 and 3.12). It can't be ported into this PR, because the conflicting string lives in the other repo's 2. This PR only edits two dependency lines in Generated by Claude Code |
What
In
pyproject.toml, switches the two vhlab dependencies fromgit+URLs to GitHub archive (zip) URLs:The other
git+dependencies (did,ndr,ndi-compress,ndi-ontology) are unchanged here; they are a later step.ndi_install.py,README.mdandtutorials/README.md(the developergit clonepath) are also untouched and will be handled separately.Why
We are building a double-click installer based on
uv tool install. uv (and pip) shell out to a systemgitforgit+URLs, and many end users (especially on Windows/macOS) have no git. Archive URLs are fetched over plain HTTPS, so git is no longer needed.When two packages depend on the same package by direct URL, uv/pip require the URL strings to be identical.
ndiandvhlab-library-pythonboth depend onvhlab-toolbox-python. The companion PR switches vhlab-library-python to exactly the same string: VH-Lab/vhlab-library-python#12Until both are merged, installing
ndifails with a conflicting-URL error forvhlab-toolbox-python. If vhlab-library merges first, NDI's install onmainbreaks; if this PR merges first, it breaks the other way. Please merge them back-to-back. CI on this PR may show that conflict until vhlab-library-python#12 is merged.Note on caching
uv caches archive URLs, so updating an existing install later to pick up new
maincommits needs--reinstall/--refresh(e.g.uv tool install --reinstall ...).Testing
black --check src/ tests/andruff check src/ tests/: pass.vhlab-toolbox-pythonURL is byte-identical to the one in vhlab-library-python#12 (both itspyproject.tomland itsrequirements.txt).github.com/.../archive/...downloads, sondicould not be installed here andpytestcould not run. This PR relies on CI for the real install + test run.🤖 Generated with Claude Code
https://claude.ai/code/session_01UMLSmPQyLVBfBC52mCahmk
Generated by Claude Code