jonkeane commented on PR #38052:
URL: https://github.com/apache/arrow/pull/38052#issuecomment-1750781795
Thanks for merging.
For posterity only:
> (It floated in last night that the `system("...", ignore.stdout = TRUE) ==
1` might be sufficient if the return code is what's causing the warning. Either
way!)
We actually do need to inter to get the output. `system("...", ignore.stdout
= TRUE) == 1 ` will be 1 if you're on x86 (because the command failed to find
`sysctl.proc_translated`), but 0 on an M* processor _regardless if you're under
emulation or not_ since the command succeeded at finding the property and
printing 0 or 1 to the console depending on if you're in emulation or not.
There might be a cleaner `system2` incantation, but I didn't want to drift too
far from what was there.
--
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]
