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


##########
r/vignettes/developers/setup.Rmd:
##########
@@ -40,13 +40,15 @@ 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.
+implementation. Because the R package integrates tightly with Arrow C++,
+it typically requires a dedicated copy of the library (i.e., it is usually
+not possible to link to a system version of libarrow during development).
 
-## Option 1: Using the bundled build with nightly libarrow binaries
+## Option 1: Using nightly libarrow binaries
 
-On Linux and MacOS, you can use the same workflow you might use for another
-package that contains compiled code (e.g., `R CMD INSTALL . --preclean` from
-the shell, `devtools::load_all()` from an R prompt, or `Install & Restart` from
+On Linux, MacOS, and Windows you can use the same workflow you might use for 
another
+package that contains compiled code (e.g., `R CMD INSTALL .` from
+a terminal, `devtools::load_all()` from an R prompt, or `Install & Restart` 
from
 RStudio). If the `libarrow` directory is not populated, the configure script 
will
 attempt to download the latest nightly libarrow binary, extract it to the
 `arrow/r/libarrow` directory (MacOS, Linux) or `arrow/r/windows/libarrow`

Review Comment:
   ```suggestion
   `arrow/r/libarrow` directory (MacOS, Linux) or `arrow/r/windows`
   ```
   I think that path was incorrect? In both of these an 'arrow-<version>' dir 
will be created and hold the actual lib. There can also be multiple versions, 
especially now with getting the newest nightly. Configure will pick up the 
newest one.



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