nealrichardson commented on pull request #11001: URL: https://github.com/apache/arrow/pull/11001#issuecomment-909596512
> * Do you want to include all of the downloaded files (87 MB), or just the ones an R build could possibly use (55 MB)? > > * edit: Currently including all Ok with me, we can revisit later if you or someone else needs a thinner fat build > * Do you still want to be able to run `download_optional_dependencies` from within an installed R package? If not, we can use the copy of `download_dependencies.sh` that's in `tools/cpp/thirdparty/` (we're currently making another copy for `inst/` so it's available at runtime). > > * edit: Deleted the function and these copies of those two files 👍 > * Should we bundle cmake while we're at it? This might be convenient, but is a bit of scope creep. > > * edit: I didn't, but let me know if you think I should. No I don't think we should bundle cmake, you can get that from a package manager easy enough, or just download one more file. The (or one of the) trouble with the C++ dependencies is that you can't get many of them as system dependencies so we have to build them ourselves. -- 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]
