Skip to content

Commit 1c4f14b

Browse files
bpmctclaude
andcommitted
fix: ensure unzip is available for Bun installation
This fix explicitly installs unzip in the same command chain where it's used for the Bun installation process. Even though unzip is installed earlier in the Dockerfile, this ensures it's available at the exact point it's needed, avoiding potential issues in CI build environments like Depot. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bf07a14 commit 1c4f14b

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

dogfood/coder/Dockerfile

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -334,11 +334,14 @@ RUN curl --silent --show-error --location --output /usr/local/bin/cloud_sql_prox
334334
curl --silent --show-error --location --output /usr/local/bin/cosign "https://github.com/sigstore/cosign/releases/download/v${COSIGN_VERSION}/cosign-linux-amd64" && \
335335
chmod a=rx /usr/local/bin/cosign && \
336336
# Install Bun JavaScript runtime to /usr/local/bin
337+
# Ensure unzip is installed right before using it
338+
apt-get update && apt-get install -y unzip && \
337339
curl --silent --show-error --location --fail "https://github.com/oven-sh/bun/releases/download/bun-v${BUN_VERSION}/bun-linux-x64.zip" --output /tmp/bun.zip && \
338340
unzip -q /tmp/bun.zip -d /tmp && \
339341
mv /tmp/bun-linux-x64/bun /usr/local/bin/ && \
340342
chmod a=rx /usr/local/bin/bun && \
341-
rm -rf /tmp/bun.zip /tmp/bun-linux-x64
343+
rm -rf /tmp/bun.zip /tmp/bun-linux-x64 && \
344+
apt-get clean && rm -rf /var/lib/apt/lists/*
342345

343346
# We use yq during "make deploy" to manually substitute out fields in
344347
# our helm values.yaml file. See https://github.com/helm/helm/issues/3141

0 commit comments

Comments
 (0)