[libcxx-commits] [libcxx] [libcxx] Include python3-yaml and rsync in container (PR #194182)

[via libcxx-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

rsync is needed for installing the kernel headers for the libc build. The yaml python package is needed for libc's hdrgen. This means we no longer have to install these utilities at runtime.

They should be small enough relative to the existing container image size to not really have an impact in that regard.

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


1 Files Affected:

- (modified) libcxx/utils/ci/docker/linux-builder-base.dockerfile (+2) 


``````````diff
diff --git a/libcxx/utils/ci/docker/linux-builder-base.dockerfile b/libcxx/utils/ci/docker/linux-builder-base.dockerfile
index 90c2b0554e276..574bb4ae25f4a 100644
--- a/libcxx/utils/ci/docker/linux-builder-base.dockerfile
+++ b/libcxx/utils/ci/docker/linux-builder-base.dockerfile
@@ -76,6 +76,8 @@ RUN sudo apt-get update \
         python3-setuptools \
         python3-psutil \
         python3-venv \
+        python3-yaml \
+        rsync \
         software-properties-common \
         swig \
         unzip \

``````````

</details>


https://github.com/llvm/llvm-project/pull/194182