jonkeane commented on a change in pull request #11870:
URL: https://github.com/apache/arrow/pull/11870#discussion_r763066581
##########
File path: r/tools/nixlibs.R
##########
@@ -77,7 +77,7 @@ download_binary <- function(os = identify_os()) {
cat("**** If installation fails, retry after installing those system
requirements\n")
}
} else {
- cat(sprintf("*** No C++ binaries found for %s\n", os))
+ cat(sprintf("*** No Arrow binary found for version %s on %s\n", VERSION,
os))
Review comment:
I like adding the version info into this, that's super helpful. I think
we probably do want some reference to C++ or libarrow here though to be sure
the person doesn't think that it's the R package binary we're talking about at
this point
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
