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

Sylwester Lachiewicz resolved THRIFT-5886.
------------------------------------------
    Resolution: Duplicate

Closing as a duplicate of THRIFT-5887, which carries the same title and was 
filed by the same reporter on the same day. THRIFT-5887 is resolved with 
fixVersion 0.23.0.

The fix is in place: {{CMakeLists.txt}} now uses {{list(PREPEND 
CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/build/cmake")}}, landed in 
{{458eb327a}} under THRIFT-5887.

_This comment was created with AI assistance._

> build/cmake/ should be prepended (not appended) to CMAKE_MODULE_PATH
> --------------------------------------------------------------------
>
>                 Key: THRIFT-5886
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5886
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>    Affects Versions: 0.22.0
>            Reporter: Kouhei Sutou
>            Priority: Minor
>
> https://github.com/apache/thrift/blob/9cff8b9f5b217c143c75299939c32573ccd4c9c3/CMakeLists.txt#L38
>  appends {{build/cmake/}} to {{CMAKE_MODULE_PATH}}.
> If we build Apache Thrift by {{FetchContent}} 
> https://cmake.org/cmake/help/latest/module/FetchContent.html, 
> {{CMAKE_MODULE_PATH}} may have some extra paths. If these extra paths have 
> the same file name in  
> https://github.com/apache/thrift/tree/master/build/cmake, our files aren't 
> used.
> For example, both of Apache Arrow and Apache Thrift have 
> {{DefineOptions.cmake}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to