From d7d31c01c76a8951c33b3ea41e644bbbe7a0cff1 Mon Sep 17 00:00:00 2001 From: DavidKorczynski Date: Fri, 13 Dec 2024 22:01:47 +0000 Subject: [PATCH] chronos: fix PATH setting (#12845) 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 --- infra/experimental/chronos/prepare-ccache | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/infra/experimental/chronos/prepare-ccache b/infra/experimental/chronos/prepare-ccache index e44e1d7cac4b..3b0f56719680 100755 --- a/infra/experimental/chronos/prepare-ccache +++ b/infra/experimental/chronos/prepare-ccache @@ -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"