nealrichardson opened a new issue, #39620:
URL: https://github.com/apache/arrow/issues/39620

   ### Describe the enhancement requested
   
   There is some drift/inconsistency/confusion in the various env vars we 
expose to control the R package build. See discussion leading up to and 
including https://github.com/apache/arrow/pull/39602#issuecomment-1892711063. 
   
   Proposal: 
   
   * `FORCE_BUNDLED_BUILD` should be deprecated in favor of `LIBARROW_BUILD`. I 
don't think we recommend it to people, it's more for our testing.
   * `LIBARROW_BUILD=true` is equivalent to `LIBARROW_BINARY=false`. If 
`LIBARROW_BUILD=true` and `LIBARROW_BINARY` is something other than unset or 
`false`, raise an error.
   * Set `download_libarrow_ok` based on `LIBARROW_BINARY=false`, not 
`LIBARROW_DOWNLOAD`. Remove references to `LIBARROW_DOWNLOAD`. (Confirm if this 
`download_libarrow_ok` is necessary, or if `LIBARROW_BINARY=false` is already 
handled correctly.)
   * If we want something explicit about preventing downloading, that's 
`TEST_OFFLINE_BUILD` (as of #39602, it blocks all downloading). If that's a 
feature we want to advertise, we could rename that something that's not `TEST_`.
   
   ### Component(s)
   
   R


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