assignUser commented on code in PR #39625:
URL: https://github.com/apache/arrow/pull/39625#discussion_r1452832102
##########
ci/scripts/r_docker_configure.sh:
##########
@@ -91,8 +91,9 @@ if [ -f
"${ARROW_SOURCE_HOME}/ci/scripts/r_install_system_dependencies.sh" ]; th
"${ARROW_SOURCE_HOME}/ci/scripts/r_install_system_dependencies.sh"
fi
-# Install rsync for bundling cpp source and curl to make sure it is installed
on all images
-$PACKAGE_MANAGER install -y rsync curl
+# Install rsync for bundling cpp source and curl to make sure it is installed
on all images,
+# cmake is now a listed sys req.
+$PACKAGE_MANAGER install -y rsync cmake curl
Review Comment:
We should(tm) have new enough cmake the only issue would be centos 7 but as
we source devtool set there that should work as well. But we will see^^
--
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]
