[llvm] [GitHub][CI] Move PATH setting into base image for tooling containers (PR #166826)
Tom Stellard via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 6 10:54:57 PST 2025
https://github.com/tstellar created https://github.com/llvm/llvm-project/pull/166826
This eliminate some redundant code.
>From 7419dabc5ab3f530a872307f270ef25de8bc1780 Mon Sep 17 00:00:00 2001
From: Tom Stellard <tstellar at redhat.com>
Date: Tue, 4 Nov 2025 22:14:02 +0000
Subject: [PATCH] [GitHub][CI] Move PATH setting into base image for tooling
containers
This eliminate some redundant code.
---
.../workflows/containers/github-action-ci-tooling/Dockerfile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/.github/workflows/containers/github-action-ci-tooling/Dockerfile b/.github/workflows/containers/github-action-ci-tooling/Dockerfile
index 8aaa2e88f2bab..707bdb309b789 100644
--- a/.github/workflows/containers/github-action-ci-tooling/Dockerfile
+++ b/.github/workflows/containers/github-action-ci-tooling/Dockerfile
@@ -22,6 +22,7 @@ RUN apt-get update && \
FROM docker.io/library/ubuntu:24.04 AS base
ENV LLVM_SYSROOT=/opt/llvm
+ENV PATH=${LLVM_SYSROOT}/bin:${PATH}
# Need nodejs for some of the GitHub actions.
# Need git for git-clang-format.
@@ -53,7 +54,6 @@ COPY --from=llvm-downloader /llvm-extract/LLVM-${LLVM_VERSION}-Linux-X64/bin/cla
/llvm-extract/LLVM-${LLVM_VERSION}-Linux-X64/bin/git-clang-format \
${LLVM_SYSROOT}/bin/
-ENV PATH=${LLVM_SYSROOT}/bin:${PATH}
# Install dependencies for 'pr-code-format.yml' job
COPY llvm/utils/git/requirements_formatting.txt requirements_formatting.txt
@@ -77,7 +77,6 @@ COPY clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py ${LLVM_SYSROOT}/bin/cl
RUN ln -s ${LLVM_SYSROOT}/bin/clang-${LLVM_VERSION_MAJOR} ${LLVM_SYSROOT}/bin/clang && \
ln -s ${LLVM_SYSROOT}/bin/clang ${LLVM_SYSROOT}/bin/clang++
-ENV PATH=${LLVM_SYSROOT}/bin:${PATH}
RUN apt-get update && \
DEBIAN_FRONTEND=noninteractive apt-get install -y \
More information about the llvm-commits
mailing list