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

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


The following commit(s) were added to refs/heads/master by this push:
     new 8a36f0f6cb ARROW-16683: [C++] Add missing dependency to bundled gflags 
target
8a36f0f6cb is described below

commit 8a36f0f6cb385c88b637f479cc38b7e51d45c7e7
Author: Sutou Kouhei <[email protected]>
AuthorDate: Mon May 30 18:52:22 2022 +0200

    ARROW-16683: [C++] Add missing dependency to bundled gflags target
    
    Closes #13256 from kou/cpp-gflags-dependency
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Antoine Pitrou <[email protected]>
---
 cpp/cmake_modules/ThirdpartyToolchain.cmake | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake 
b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index ea9cb6d23d..6fed574619 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -1379,6 +1379,7 @@ macro(build_gflags)
   add_dependencies(toolchain gflags_ep)
 
   add_thirdparty_lib(gflags::gflags_static STATIC ${GFLAGS_STATIC_LIB})
+  add_dependencies(gflags::gflags_static gflags_ep)
   set(GFLAGS_LIBRARY gflags::gflags_static)
   set_target_properties(${GFLAGS_LIBRARY}
                         PROPERTIES INTERFACE_COMPILE_DEFINITIONS 
"GFLAGS_IS_A_DLL=0"

Reply via email to