Skip to content

Commit cb2fa43

Browse files
committed
build: merge docker run instructions
1 parent 1f07329 commit cb2fa43

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

Dockerfile

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,20 +29,18 @@ RUN \
2929
bison \
3030
flex
3131

32-
# install UV and create .venv
33-
RUN curl -LsSf https://astral.sh/uv/$(cat ./requirement-uv.txt | awk -F'==' '{print $2}' | tr -d ' \n')/install.sh | sh && \
34-
. $HOME/.local/bin/env && \
35-
uv sync --frozen --no-install-project
36-
37-
# install shared tools from tools.json;
32+
# install UV, .venv and shared tools;
3833
#
39-
# The idea here is that we pre-pack all the tools into the Docker/Podman image that can be used for any device:
34+
# The main idea here is that we pre-pack all the tools into the Docker/Podman image that can be used for any device:
4035
# (toolchains, binutils -- everything except device-specific kernel source);
4136
#
4237
# This significantly reduces the total build time, as each time we make a build call for a device,
4338
# only device-specific kernel source is being downloaded into the container.
4439
#
45-
RUN uv run ${WDIR}/builder/utils/bridge.py --shared
40+
RUN curl -LsSf https://astral.sh/uv/$(cat ./requirement-uv.txt | awk -F'==' '{print $2}' | tr -d ' \n')/install.sh | sh && \
41+
. $HOME/.local/bin/env && \
42+
uv sync --frozen --no-install-project && \
43+
uv run ${WDIR}/builder/utils/bridge.py --shared
4644

4745
# activate .venv
4846
CMD [ "source", ".venv/bin/activate" ]

0 commit comments

Comments
 (0)