kou commented on code in PR #49265:
URL: https://github.com/apache/arrow/pull/49265#discussion_r2808578988
##########
ci/docker/debian-13-cpp.dockerfile:
##########
@@ -89,6 +89,9 @@ RUN apt-get update -y -q && \
tzdata \
tzdata-legacy \
zlib1g-dev && \
+ if [ ${arch} = "i386" ]; then \
Review Comment:
I'm OK with installing Rust here.
BTW, how about always installing Rust instead of checking `${arch} = "i386"`?
```diff
diff --git a/ci/docker/debian-13-cpp.dockerfile
b/ci/docker/debian-13-cpp.dockerfile
index fe947db025..928443ee64 100644
--- a/ci/docker/debian-13-cpp.dockerfile
+++ b/ci/docker/debian-13-cpp.dockerfile
@@ -42,6 +42,7 @@ RUN apt-get update -y -q && \
apt-get update -y -q && \
apt-get install -y -q --no-install-recommends \
autoconf \
+ cargo \
ccache \
clang-${llvm} \
cmake \
@@ -86,6 +87,7 @@ RUN apt-get update -y -q && \
python3-venv \
rapidjson-dev \
rsync \
+ rust \
tzdata \
tzdata-legacy \
zlib1g-dev && \
```
I know that we don't need Rust for the armd64 platform but no `${arch}`
check will simplify our Dockerfile.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
