[GitHub] [arrow] nealrichardson commented on a diff in pull request #34048: GH-32292: [R][Packaging] Use binaries build on CentOS 7 for Ubuntu < 22.04

["nealrichardson (via GitHub)" <gi...@apache.org>]

nealrichardson commented on code in PR #34048:
URL: https://github.com/apache/arrow/pull/34048#discussion_r1097550745


##########
ci/scripts/r_install_system_dependencies.sh:
##########
@@ -37,10 +37,10 @@ if [ "$ARROW_S3" == "ON" ] || [ "$ARROW_GCS" == "ON" ] || [ "$ARROW_R_DEV" == "T
   # Install curl and OpenSSL for S3/GCS support
   case "$PACKAGE_MANAGER" in
     apt-get)
-      apt-get install -y libcurl4-openssl-dev libssl-dev
+      apt-get install -y libcurl4-openssl-dev libssl-dev pkg-config

Review Comment:
   How common is it to be on ubuntu and not have pkg-config installed? Does this mean we would require pkg-config in order for the binary to be used?



-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org