Skip to content

Commit e2e1812

Browse files
committed
fix: container now copies managed python into runtime
1 parent c8fc7b7 commit e2e1812

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

.devcontainer/devcontainer.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
"remoteEnv": {
99
// Allow X11 apps to run inside the container
1010
"DISPLAY": "${localEnv:DISPLAY}",
11-
// We put venv on a different volume to cache so have to copy
12-
"UV_LINK_MODE": "copy",
1311
// Do the equivalent of "activate" the venv so we don't have to "uv run" everything
1412
"PATH": "/workspaces/${localWorkspaceFolderBasename}/.venv/bin:${containerEnv:PATH}"
1513
},

template/Dockerfile.jinja

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@ WORKDIR /app
1616
COPY . /app
1717
RUN chmod o+wrX .
1818

19+
# Tell uv sync to install python in a known location so we can copy it out later
20+
ENV UV_PYTHON_INSTALL_DIR=/python
21+
1922
# Sync the project without its dev dependencies
2023
RUN --mount=type=cache,target=/root/.cache/uv \
2124
uv sync --locked --no-editable --no-dev
@@ -50,6 +53,9 @@ FROM ubuntu:noble AS runtime
5053
# some-library \
5154
# && apt-get dist-clean
5255

56+
# Copy the python installation from the build stage
57+
COPY --from=build /python /python
58+
5359
# Copy the environment, but not the source code
5460
COPY --from=build /app/.venv /app/.venv
5561
ENV PATH=/app/.venv/bin:$PATH

0 commit comments

Comments
 (0)