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


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

Review Comment:
   That's true...I would still characterize the workflow as `R CMD INSTALL .` 
until you get a build error, and *then* delete the `libarrow` directory and do 
`R CMD INSTALL . --preclean`. When you're iterating on the R package you 
usually don't want `--preclean` because often you just changed R code and can 
skip the compile alltogether.



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