[jira] [Resolved] (ARROW-17868) [C++][Python] Keep and deprecate ARROW_PYTHON CMake option for backward compatibility

["Kouhei Sutou (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ARROW-17868?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kouhei Sutou resolved ARROW-17868.
----------------------------------
    Fix Version/s: 10.0.0
       Resolution: Fixed

Issue resolved by pull request 14273
[https://github.com/apache/arrow/pull/14273]

> [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
>             Fix For: 10.0.0
>
>          Time Spent: 3h 40m
>  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)