Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[libc++] Bump the version of CMake built in the CI Docker image #120240

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

ldionne
Copy link
Member

@ldionne ldionne commented Dec 17, 2024

This will allow using the $<LINK_LIBRARY> generator expression in some of our configurations. We should separately pursue officially bumping the minimum CMake version across all LLVM so we can use this feature more widely.

This allows using the $<LINK_LIBRARY> generator expression in some of
our configurations. We should separately pursue officially bumping
the minimum CMake version across all LLVM so we can use this feature
more widely.
@ldionne ldionne requested a review from a team as a code owner December 17, 2024 15:00
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Dec 17, 2024
@llvmbot
Copy link
Member

llvmbot commented Dec 17, 2024

@llvm/pr-subscribers-libcxx

Author: Louis Dionne (ldionne)

Changes

This allows using the $<LINK_LIBRARY> generator expression in some of our configurations. We should separately pursue officially bumping the minimum CMake version across all LLVM so we can use this feature more widely.


Full diff: https://github.com/llvm/llvm-project/pull/120240.diff

1 Files Affected:

  • (modified) libcxx/utils/ci/Dockerfile (+1-1)
diff --git a/libcxx/utils/ci/Dockerfile b/libcxx/utils/ci/Dockerfile
index dbecfd9fd3325d..d0144908f098bd 100644
--- a/libcxx/utils/ci/Dockerfile
+++ b/libcxx/utils/ci/Dockerfile
@@ -160,7 +160,7 @@ EOF
 RUN <<EOF
     # Install a recent CMake
     set -e
-    wget https://github.com/Kitware/CMake/releases/download/v3.21.1/cmake-3.21.1-linux-x86_64.sh -O /tmp/install-cmake.sh
+    wget https://github.com/Kitware/CMake/releases/download/v3.24.4/cmake-3.24.4-linux-x86_64.sh -O /tmp/install-cmake.sh
     sudo bash /tmp/install-cmake.sh --prefix=/usr --exclude-subdir --skip-license
     rm /tmp/install-cmake.sh
 EOF

@ldionne ldionne changed the title [libc++] Bump the version of CMake used in our Docker containers [libc++] Bump the version of CMake built in the CI Docker image Dec 17, 2024
@ldionne ldionne merged commit d8a0709 into llvm:main Dec 17, 2024
17 of 20 checks passed
@ldionne ldionne deleted the review/bump-cmake-in-libcxx-CI branch December 17, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants