Skip to content

Commit

Permalink
feat: upgrade host llvm from 18 to 19
Browse files Browse the repository at this point in the history
  • Loading branch information
antonbaliasnikov committed Nov 11, 2024
1 parent ee8d04f commit bb3c01d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions images/zksync-llvm-runner/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,8 @@ RUN apt-get update && \
&& rm -rf /var/lib/apt/lists/*

# Install LLVM
RUN curl https://apt.llvm.org/llvm.sh -sSf | bash -s -- 18 all && \
ENV LLVM_VERSION=19
RUN curl https://apt.llvm.org/llvm.sh -sSf | bash -s -- ${LLVM_VERSION} all && \
rm -rf /var/lib/apt/lists/*

# Install Python 3.11
Expand Down Expand Up @@ -76,9 +77,8 @@ ENV RUSTUP_HOME=/usr/local/rustup \
RUN curl https://sh.rustup.rs -sSf | bash -s -- -y

# Set required environment variables
ENV PATH=/usr/lib/llvm-18/bin:${PATH} \
LD_LIBRARY_PATH=/usr/lib/llvm-18/lib:${LD_LIBRARY_PATH} \
LLVM_VERSION=18 \
ENV PATH=/usr/lib/llvm-${LLVM_VERSION}/bin:${PATH} \

Check warning on line 80 in images/zksync-llvm-runner/Dockerfile

View workflow job for this annotation

GitHub Actions / release

Variables should be defined before their use

UndefinedVar: Usage of undefined variable '$LD_LIBRARY_PATH' More info: https://docs.docker.com/go/dockerfile/rule/undefined-var/
LD_LIBRARY_PATH=/usr/lib/llvm-${LLVM_VERSION}/lib:${LD_LIBRARY_PATH} \
CI_RUNNING=true

# Replace default libm.a which is a linker script on x86_64 to an actual lib implementation
Expand Down

0 comments on commit bb3c01d

Please sign in to comment.