jorisvandenbossche opened a new pull request, #41494:
URL: https://github.com/apache/arrow/pull/41494

   ### Rationale for this change
   
   Currently, when building pyarrow from source, one needs to manually enable 
the optional components through setting `PYARROW_WITH_...` environment 
variables. However, we could also make a default choice of components based on 
which ones where enabled in the Arrow C++ build.
   
   ### What changes are included in this PR?
   
   Set defaults for the various `PYARROW_BUILD_<component>` based on the 
`ARROW_<component>` setting. Keep the current `PYARROW_WITH_<component>` 
environment variables working to allow to override this default.
   
   One minor unfortunate consequence is that if you change your Arrow C++ build 
(eg enable a new component), you need to do a clean python build, otherwise 
CMake is still getting the old options (those option defaults are cached? Not 
sure if there is a way to solve this)
   
   ### Are these changes tested?
   
   <!--
   We typically require tests for all PRs in order to:
   1. Prevent the code from being accidentally broken by subsequent changes
   2. Serve as another way to document the expected behavior of the code
   
   If tests are not included in your PR, please explain why (for example, are 
they covered by existing tests)?
   -->
   
   ### Are there any user-facing changes?
   
   No


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to