nealrichardson commented on a change in pull request #10849:
URL: https://github.com/apache/arrow/pull/10849#discussion_r680894618



##########
File path: dev/tasks/docker-tests/azure.linux.yml
##########
@@ -45,6 +45,15 @@ jobs:
       git -C arrow submodule update --init --recursive
     displayName: Clone arrow
 
+  # only needed for r-lto job to use until 
https://github.com/wch/r-debug/pull/22 is merged
+  - script: |
+      git clone https://github.com/nealrichardson/r-debug.git
+      git checkout lto
+      cd r-debug
+      ./buildall.sh

Review comment:
       A possibly faster way to do this is to add a new job like for the 
r-sanitizer, and add dockerfile like 
https://github.com/nealrichardson/r-debug/blob/lto/r-debug-5/Dockerfile:
   
   ```
   FROM wch1/r-debug
   
   # RDlto: R built with --enable-lto
   RUN wget 
https://raw.githubusercontent.com/nealrichardson/r-debug/lto/r-devel/buildR.sh
   RUN buildR.sh lto
   RUN RDlto -q -e 'install.packages("pak", repos = 
"https://r-lib.github.io/p/pak/dev";)'
   RUN RDlto -q -e 'pak::pkg_install(c("devtools", "Rcpp", "roxygen2", 
"testthat", "memoise", "rmarkdown"))'
   ```
   




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