Add uv and bloc using uv tool install #27
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "agent/add-bloc-uv-tool"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Evaluation
Brad asked me to evaluate the best option for installing bloc in sandbox containers. After reviewing the options, I recommend:
✅ Install uv + use
uv tool installWhy This Approach
/root/.local/bin/, not/root/.cargo/bin/uv tool installis the recommended way to install CLI tools globallyImplementation
vs. git clone + pip install
PR #26 uses
git clone + uv pip install --system. This works, but:/root/.cargo/bin/(should be/root/.local/bin/)Testing
Once merged and the image is rebuilt:
Auth is automatic via rbw (FORGEJO_TOKEN already configured).
Related
@agent sorry i didnt see your note before merging the other PR. i think your approach is correct. can you update from main and then add your change?
Updated from main and removed the duplicate installation. The branch now has only the
uv tool installapproach:This approach provides better isolation (each tool gets its own venv) and is more idiomatic for CLI tool installation.
@agent i notice we need to handle this differently for build workflow
The error you're seeing is from the old code on
main(commita0d84cd) that tried to move from/root/.cargo/bin/uv, but the uv installer actually puts the binary in/root/.local/bin/uv.The current PR branch (
78178e2) already has the fix — it correctly uses/root/.local/bin/uv. I cleaned up the duplicate installation and kept only theuv tool installapproach.If you're seeing this error from a recent build:
docker builder prune)agent/add-bloc-uv-toolbranch, notmainThe current Dockerfile on this branch should work.