jonkeane commented on code in PR #41019:
URL: https://github.com/apache/arrow/pull/41019#discussion_r1552590380
##########
r/R/arrow-package.R:
##########
@@ -182,37 +182,50 @@ configure_tzdb <- function() {
.onAttach <- function(libname, pkgname) {
# Just to be extra safe, let's wrap this in a try();
# we don't want a failed startup message to prevent the package from loading
- try({
- # On macOS only, Check if we are running in under emulation, and warn this
will not work
- if (on_rosetta()) {
- packageStartupMessage(
- paste(
- "Warning:",
- " It appears that you are running R and Arrow in emulation (i.e.
you're",
- " running an Intel version of R on a non-Intel mac). This
configuration is",
- " not supported by arrow, you should install a native (arm64) build
of R",
- " and use arrow with that. See
https://cran.r-project.org/bin/macosx/";,
- "",
- sep = "\n"
+ try(
+ {
Review Comment:
styler apparently demands this now?. I can back it out if we don't like it,
but came from `make doc` so we'll be fighting it if we do.
--
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]
