Skip to content

Build Docker image with uv.lock#171

Open
JosephMarinier wants to merge 3 commits into
mainfrom
joseph/build-docker-image-with-uv-lock
Open

Build Docker image with uv.lock#171
JosephMarinier wants to merge 3 commits into
mainfrom
joseph/build-docker-image-with-uv-lock

Conversation

@JosephMarinier

@JosephMarinier JosephMarinier commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator
  • Build Docker image with uv.lock
  • Then, I asked Claude Code to review the whole Dockefile, and it suggested a few more changes:
    • Copy eva dist-info into runtime image alongside the package
      The runtime stage overlays the eva package from the builder, but was missing its dist-info directory. Without it, importlib.metadata calls (e.g. version lookups) fail and package inspection tools report eva as not installed.
    • Drop build-essential and git from the builder stage
      The builder only runs uv pip install --no-deps . on a pure Python package (hatchling, no native extensions). Neither compiler toolchain nor git is needed for that step; they were only required in the deps stage where third-party packages are compiled.

JosephMarinier and others added 3 commits June 30, 2026 15:00
The runtime stage overlays the eva package from the builder, but was
missing its dist-info directory. Without it, importlib.metadata calls
(e.g. version lookups) fail and package inspection tools report eva
as not installed.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The builder only runs `uv pip install --no-deps .` on a pure Python
package (hatchling, no native extensions). Neither compiler toolchain
nor git is needed for that step; they were only required in the deps
stage where third-party packages are compiled.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@JosephMarinier JosephMarinier self-assigned this Jun 30, 2026
@JosephMarinier JosephMarinier force-pushed the joseph/build-docker-image-with-uv-lock branch from 4e980f7 to b7ace15 Compare June 30, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants