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

wesm 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 16ddb12  ARROW-1914: [C++] Fix build dependency for GPU support build
16ddb12 is described below

commit 16ddb127c5e6e51d80aa24d94dffded393db78a9
Author: Kouhei Sutou <[email protected]>
AuthorDate: Tue Dec 12 15:08:54 2017 -0500

    ARROW-1914: [C++] Fix build dependency for GPU support build
    
    "make -j" may cause build error:
    
        [100%] Built target gflags_nothreads_static
        Install the project...
        -- Install configuration: "RELEASE"
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/lib/cmake/gflags/gflags-config.cmake
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/lib/cmake/gflags/gflags-config-version.cmake
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/lib/cmake/gflags/gflags-targets.cmake
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/lib/cmake/gflags/gflags-targets-release.cmake
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/bin/gflags_completions.sh
        -- Installing: 
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/build/gflags_ep-prefix/src/gflags_ep/lib/pkgconfig/gflags.pc
        -- Installing: 
/home/kou/.cmake/packages/gflags/fb801def37c922433975cbfefb3aa08d
        [ 26%] Completed 'gflags_ep'
        [ 55%] Building C object CMakeFiles/brotlienc.dir/enc/literal_cost.c.o
        [ 26%] Built target gflags_ep
        Scanning dependencies of target arrow_gpu_objlib
        [ 26%] Building CXX object 
src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/cuda_arrow_ipc.cc.o
        [ 59%] Building C object CMakeFiles/brotlienc.dir/enc/memory.c.o
        
/tmp/arrow-0.8.0.GLyu7/apache-arrow-0.8.0/cpp/src/arrow/gpu/cuda_arrow_ipc.cc:26:10:
 fatal error: arrow/ipc/Message_generated.h: No such file or directory
         #include "arrow/ipc/Message_generated.h"
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        compilation terminated.
        src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/build.make:62: recipe for 
target 'src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/cuda_arrow_ipc.cc.o' 
failed
        make[2]: *** 
[src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/cuda_arrow_ipc.cc.o] Error 1
        CMakeFiles/Makefile2:2108: recipe for target 
'src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/all' failed
        make[1]: *** [src/arrow/gpu/CMakeFiles/arrow_gpu_objlib.dir/all] Error 2
        make[1]: *** Waiting for unfinished jobs....
    
    Author: Kouhei Sutou <[email protected]>
    
    Closes #1414 from kou/cpp-fix-gpu-build-dependency and squashes the 
following commits:
    
    8fe28c7a [Kouhei Sutou] [C++] Fix build dependency for GPU support build
---
 cpp/src/arrow/gpu/CMakeLists.txt | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/cpp/src/arrow/gpu/CMakeLists.txt b/cpp/src/arrow/gpu/CMakeLists.txt
index 3ddf2c7..be47413 100644
--- a/cpp/src/arrow/gpu/CMakeLists.txt
+++ b/cpp/src/arrow/gpu/CMakeLists.txt
@@ -32,9 +32,6 @@ set(ARROW_GPU_SRCS
   cuda_memory.cc
 )
 
-add_custom_target(arrow_gpu_sources DEPENDS ${ARROW_GPU_SRCS})
-add_dependencies(arrow_gpu_sources metadata_fbs)
-
 set(ARROW_GPU_SHARED_LINK_LIBS
   arrow_shared
   ${CUDA_LIBRARIES}
@@ -43,6 +40,7 @@ set(ARROW_GPU_SHARED_LINK_LIBS
 
 ADD_ARROW_LIB(arrow_gpu
   SOURCES ${ARROW_GPU_SRCS}
+  DEPENDENCIES metadata_fbs
   SHARED_LINK_FLAGS ""
   SHARED_LINK_LIBS ${ARROW_GPU_SHARED_LINK_LIBS}
   STATIC_LINK_LIBS ""

-- 
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].

Reply via email to