alamb commented on a change in pull request #9891:
URL: https://github.com/apache/arrow/pull/9891#discussion_r607191808
##########
File path: ci/docker/conda-integration.dockerfile
##########
@@ -25,32 +25,27 @@ ARG node=14
ARG jdk=8
ARG go=1.15
+# Uninstall unused space-consuming packages
+# (XXX: it would be better not to install them, but they are used by other
+# builds which are also based on conda-cpp)
+RUN conda uninstall -q clangdev llvmdev valgrind
+
+# Install Archery and integration dependencies
COPY ci/conda_env_archery.yml /arrow/ci/
RUN conda install -q \
--file arrow/ci/conda_env_archery.yml \
numpy \
maven=${maven} \
nodejs=${node} \
openjdk=${jdk} && \
- conda clean --all
+ conda clean --all --force-pkgs-dirs
-RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
+# Install Rust and remove space-consuming docs
+RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y &&
\
+ rm -rf `find /root/.rustup/ -name html -type d`
Review comment:
@pitrou I don't think we use `rustfmt` directly, but rather it is used
by `tonic_build` https://docs.rs/tonic-build/0.4.1/tonic_build/ which is used
to compile the `protobuf` definitions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
