jonkeane commented on a change in pull request #11875:
URL: https://github.com/apache/arrow/pull/11875#discussion_r764372413



##########
File path: ci/scripts/r_docker_configure.sh
##########
@@ -77,5 +77,19 @@ if [ "$ARROW_S3" == "ON" ] || [ "$ARROW_R_DEV" == "TRUE" ]; 
then
   fi
 fi
 
+# Install patch if it doesn't already exist
+if [ ! $(command -v patch) ]; then
+  if [ "`which dnf`" ]; then
+    dnf install -y patch
+  elif [ "`which yum`" ]; then
+    yum install -y patch
+  elif [ "`which zypper`" ]; then
+    zypper install -y patch
+  else
+    apt-get update
+    apt-get install -y patch
+  fi
+fi

Review comment:
       This is not strictly necessary (see below where we disable snappy if we 
can't find `patch`), but it makes sure that we do have `patch` available so we 
do exercise this code in the sanitizer in our CI




-- 
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]


Reply via email to