This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 6a7a6ee308 GH-40432: [C++] Add missing Threads::Threads dependency to 
arrow_static (#40433)
6a7a6ee308 is described below

commit 6a7a6ee308b69c12f46f874cb3d52892e172d7b7
Author: Sutou Kouhei <[email protected]>
AuthorDate: Sun Mar 10 05:41:36 2024 +0900

    GH-40432: [C++] Add missing Threads::Threads dependency to arrow_static 
(#40433)
    
    ### Rationale for this change
    
    `libarrow.a` uses `std::mutex` and so on. So we need to link to 
`Threads::Threads`. But #39824 dropped it accidentally.
    
    ### What changes are included in this PR?
    
    Add unexpectedly dropped `Threads::Threads` dependency to `arrow_static` 
again.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    Yes.
    * GitHub Issue: #40432
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 cpp/src/arrow/CMakeLists.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cpp/src/arrow/CMakeLists.txt b/cpp/src/arrow/CMakeLists.txt
index 0bf55e38f9..b53d76e747 100644
--- a/cpp/src/arrow/CMakeLists.txt
+++ b/cpp/src/arrow/CMakeLists.txt
@@ -137,6 +137,7 @@ endif()
 if(ARROW_ENABLE_THREADING)
   list(APPEND ARROW_SHARED_PRIVATE_LINK_LIBS Threads::Threads)
   list(APPEND ARROW_STATIC_LINK_LIBS Threads::Threads)
+  list(APPEND ARROW_STATIC_INSTALL_INTERFACE_LIBS Threads::Threads)
 endif()
 
 if(NOT MSVC_TOOLCHAIN)

Reply via email to