assignUser commented on code in PR #38220:
URL: https://github.com/apache/arrow/pull/38220#discussion_r1355827243


##########
r/vignettes/developers/setup.Rmd:
##########
@@ -38,50 +38,49 @@ set -e
 set -x
 ```
 
+The Arrow R package is unique compared to other R packages that you may have
+contributed to because it builds on top of the large and feature-rich Arrow C++
+implementation.
 
-```{bash, save=run & windows, hide=TRUE}
-# For some reason CRAN Mirror goes missing in CI
-echo 'options(repos=structure(c(CRAN="https://cloud.r-project.org";)))' > 
$HOME/.Rprofile
-```
-
-Windows and macOS users who wish to contribute to the R package and
-don't need to alter libarrow (Arrow's C++ library) may be able to obtain a
-recent version of the library without building from source.
-
-### Linux
+## Option 1: Using pre-built libarrow binaries
 
-On Linux, you can download a .zip file containing libarrow from the
+On Linux and MacOS, you can download a .zip file containing libarrow from the
 [nightly repository](https://nightlies.apache.org/arrow/r/libarrow/bin/).
 
 The directory names correspond to the OpenSSL version the binaries built with:
-- "linux-openssl-1.0" (OpenSSL 1.0)
-- "linux-openssl-1.1" (OpenSSL 1.1)
-- "linux-openssl-3.0" (OpenSSL 3.0)
 
-Version numbers in that repository correspond to dates.
+- "linux-openssl-1.0" (OpenSSL 1.0, e.g., Centos7)
+- "linux-openssl-1.1" (OpenSSL 1.1, e.g., Ubuntu 18.04)
+- "linux-openssl-3.0" (OpenSSL >=3.0, e.g., MacOS or Ubuntu >= 20.04)
 
-You'll need to create a `libarrow` directory inside the R package directory 
and unzip the zip file containing the compiled libarrow binary files into it.
+Version numbers in that repository correspond to dates. Use the version with
+the highest number (i.e., that was modified most frequently).
 
-### macOS
-On macOS, you can install libarrow using [Homebrew](https://brew.sh/):
+You'll need to create a `libarrow` directory inside the R package directory and
+unzip the zip file containing the compiled libarrow binary files into it.

Review Comment:
   I think this existing one is applicable: 
https://github.com/apache/arrow/issues/27839
   
   I have the logic done, I just want to refactor *libs.R a bit as they are 
overlapping more and more and I don't want to keep copying stuff across ^^ I 
should be able to get it done today... I will open a PR with my changes 
regardless of how far I am later today so you can test/review/add to it :)



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

Reply via email to