[
https://issues.apache.org/jira/browse/ARROW-17868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated ARROW-17868:
-----------------------------------
Labels: pull-request-available (was: )
> [C++][Python] Keep and deprecate ARROW_PYTHON CMake option for backward
> compatibility
> -------------------------------------------------------------------------------------
>
> Key: ARROW-17868
> URL: https://issues.apache.org/jira/browse/ARROW-17868
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++, Python
> Reporter: Kouhei Sutou
> Assignee: Kouhei Sutou
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> ARROW-6858 removed {{ARROW_PYTHON}} CMake option because ARROW-16340 moved
> {{cpp/src/arrow/python/}} to {{python/pyarrow/src/}}. But it broke backward
> compatibility. Users who use {{-DARROW_PYTHON=ON}} needs to
> {{-DARROW_CSV=ON}}, {{-DARROW_DATASET=ON}} and so on manually.
> See also: https://github.com/apache/arrow/pull/14224#discussion_r981399130
> {quote}
> FWIW this broke my local development because of no longer including those
> (although I should probably start using presets ..)
> Now, it's probably fine to remove this now Python C++ has moved, but we do
> assume that some C++ modules are built on the pyarrow side (eg we assume that
> CSV is always built, while with the above change you need to ensure manually
> that this is done in your cmake call).
> In any case we should update the documentation at
> https://arrow.apache.org/docs/dev/developers/python.html#build-and-test to
> indicate that there are a few components required to be able to build pyarrow.
> {quote}
> Eventually, we can remove {{ARROW_PYTHON}} CMake option but we should provide
> a deprecation period before we remove {{ARROW_PYTHON}}.
> We should also mention that {{ARROW_PYTHON}} is deprecated in our
> documentation (
> https://arrow.apache.org/docs/dev/developers/python.html#build-and-test ).
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
