[GitHub] [arrow] jonkeane commented on a change in pull request #11901: ARROW-14849: [R] Update messaging in installation scripts

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

jonkeane commented on a change in pull request #11901:
URL: https://github.com/apache/arrow/pull/11901#discussion_r765087769



##########
File path: r/tools/nixlibs.R
##########
@@ -600,12 +600,15 @@ if (!file.exists(paste0(dst_dir, "/include/arrow/api.h"))) {
     # (2) Find source and build it
     src_dir <- find_local_source()
     if (!is.null(src_dir)) {
-      cat("*** Building C++ libraries\n")
+      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")

Review 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")? 




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