jonkeane commented on code in PR #41403:
URL: https://github.com/apache/arrow/pull/41403#discussion_r1582198565
##########
dev/tasks/tasks.yml:
##########
@@ -1377,9 +1377,9 @@ tasks:
template: r/azure.linux.yml
params:
r_org: rhub
- r_image: debian-gcc-devel-lto
+ r_image: gcc13
r_tag: latest
- flags: '-e NOT_CRAN=false -e INSTALL_ARGS=--use-LTO'
+ flags: '-e INSTALL_ARGS=--use-LTO'
Review Comment:
The LTO image from rhub/containers doesn't have tex installed and as cran
means we check the manual so we get errors there.
We could install tex when we need it with something like this added to
r_test.sh
```
# Install tinytex if we are acting like CRAN and there is no pdflatex already
if [ $NOT_CRAN == "false" and ! command -v pdflatex &> /dev/null ]; then
echo "pdflatex is not available, installing tinytex instead"
if [ "`which dnf`" ]; then
dnf install -y texlive-scheme-basic
elif [ "`which yum`" ]; then
yum install -y texlive-scheme-basic
elif [ "`which zypper`" ]; then
zypper install -y texlive-scheme-basic
else
apt-get update
apt-get install -y texlive-latex-base
fi
# ensure that both perl and wget are installed
$PACKAGE_MANAGER install -y texlive-latex-base
which pdflatex
which texi2dvi
exit 1
fi
```
But I'm not sure this actually gets us much. So I've removed the
`NOT_CRAN=false` for now, but can add it back in with ^^^ if there's a strong
reason to.
--
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]
