[GitHub] [arrow] karldw commented on pull request #12849: ARROW-15092: [R] Support create_package_with_all_dependencies() on non-linux systems

[GitBox <gi...@apache.org>]

karldw commented on PR #12849:
URL: https://github.com/apache/arrow/pull/12849#issuecomment-1095796331

   Okay, great! @nealrichardson, just to confirm: do you want to go with the suggestion to add `pkgbuild` as a new dependency? I added it and tweaked the bash script - let's see how tests go on this new version.
   
   --------------------
   
   Specific replies:
   
   @nealrichardson:
   
   I think `readlink` without `-f` isn't useful here, so I removed it from the chain. I also had to change the syntax a little to get things working, but I might be missing some clever bash-ism.
   
   Just to repeat @assignUser's earlier comment, the use case I have in my head is that the package is downloaded on one machine, then installed on another. For that reason, I was trying not to make too many assumptions about the build capabilities on the downloading machine. But this offline build is a pretty niche demand, and it's probably okay to ask those users to make sure they have bash available when downloading.
   
   
   @assignUser:
   
   > I noticed that there are no tests for `create_package_with_all_dependencies` so that might be something we could add in this PR.
   
   Running `create_package_with_all_dependencies` requires downloading ~100MB of files, which seemed like a pretty heavy test to run every time. I added a test for `run_download_script` that skips the actual download, but checks that the requirements are in place.
   
   
   @assignUser and @wjones127, thanks for the tips!
   


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