Skip to content

Commit

Permalink
chronos: fix PATH setting (#12845)
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 13, 2024
1 parent 9fe8453 commit d7d31c0
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 d7d31c0

Please sign in to comment.