Skip to content

Commit

Permalink
chronos: fix PATH setting
Browse files Browse the repository at this point in the history
Currently the $PATH bit is expanded, i.e. the ENV string becomes the PATH of the host system, as opposed to the docker container's PATH. We should escape the `$` to ensure we don't overwrite the full PATH
  • Loading branch information
DavidKorczynski authored Dec 12, 2024
1 parent bdf19d3 commit fc1c42c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion infra/experimental/chronos/prepare-ccache
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,5 @@
PROJECT=$1
{
echo "COPY ccache-cache/ /ccache/cache";
echo "ENV PATH=/ccache/bin:$PATH"
echo "ENV PATH=\"/ccache/bin:\$PATH\""
} >> "projects/$PROJECT/Dockerfile"

0 comments on commit fc1c42c

Please sign in to comment.