ARROW-14849: [R] Update messaging in installation scripts #11901
Conversation
|
Build failure due to an inability to download cmake, rather than related to this PR |
r/tools/nixlibs.R
Outdated
| cat("*** Building libarrow\n") | ||
| cat("*** For a faster installation, set the environment variable LIBARROW_BINARY=true before installing\n") | ||
| cat("*** Not all features are enabled by default.") | ||
| cat(" For a fully-featured installation, set the environment variable LIBARROW_MINIMAL=false before installing\n") |
There was a problem hiding this comment.
Do we instead want to offer NOT_CRAN=true for both of these (and possibly collapse it down to "for a fester, more complete installation, set the environment variable NOT_CRAN=TRUE before installing")?
There was a problem hiding this comment.
Yeah, I was wanting to be more explicit about the multiple things which were going on, but I think we can get the best of both worlds by referring to NOT_CRAN and then suggesting the user takes a look at the install vignette for more info, like we do in another place in the script.
|
Benchmark runs are scheduled for baseline = 53ab5a0 and contender = 3f6773a. 3f6773a is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.