thisisnic commented on a change in pull request #11870:
URL: https://github.com/apache/arrow/pull/11870#discussion_r763847305
##########
File path: r/tools/nixlibs.R
##########
@@ -77,7 +80,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 C++ binary found for version %s on %s\n",
VERSION, os))
Review comment:
```suggestion
cat(sprintf("*** No libarrow binary found for version %s on %s\n",
VERSION, os))
```
Small change - feel free to reject - but I've been referring to Arrow C++ as
`libarrow` in a lot of the docs + it also then matches up with the env var.
--
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]
