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



##########
File path: dev/tasks/tasks.yml
##########
@@ -941,6 +941,13 @@ tasks:
   test-r-devdocs:
     ci: github
     template: r/github.devdocs.yml
+    
+  test-r-depsource-auto:
+    ci: github
+    template: docker-tests/github.linux.yml
+    params:
+      flags: '-e ARROW_DEPENDENCY_SOURCE=AUTO -e ARROW_SOURCE_HOME="/arrow" -e 
LIBARROW_BUILD=TRUE'
+      image: ubuntu-r

Review comment:
       ```
     test-r-depsource-auto:
       ci: azure
       template: r/azure.linux.yml
       params:
         r_org: rocker
         r_image: r-base
         r_tag: latest
         flags: '-e ARROW_DEPENDENCY_SOURCE=AUTO'
   ```
   
   Then it looks like you'll need to add `{{ flags|default("") }}` around L53 
in dev/tasks/r/azure.linux.yml so that the job picks them up since that 
parameter isn't defined in that template yet. (I think there's some refactoring 
we can do here to clean up these templates, but that's for another time.)




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