jonkeane commented on code in PR #40244: URL: https://github.com/apache/arrow/pull/40244#discussion_r1502973257
########## ci/scripts/r_docker_configure.sh: ########## @@ -95,5 +95,18 @@ fi # cmake is now a listed sys req. $PACKAGE_MANAGER install -y rsync cmake curl +# Install libcurl (needed to compile curl, which is needed by rcmdcheck); need texlive for CRAN checks. +# https://github.com/r-hub/containers/issues/58 +# The apt-get based images (rhub/clang17, rhub/clang18) are the only ones that seem to lack texlive right now: +# https://github.com/r-hub/containers/issues/23 +case "$PACKAGE_MANAGER" in + apt-get) + apt-get install -y libcurl4-openssl-dev texlive-latex-base texlive-fonts-extra Review Comment: I've opened an issue against the rhub/containers repo to add in libcurl4-openssl. But we need this to be installed for curl to install on this machine. texlive installing is also probably not what we want here, but it was necessary to get the build to be clean. We might consider turning off pdf manual checking on these if tex isn't available or installing tinytex in some other way. -- 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]
