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 341f1b2  ARROW-4585: [C++] Add protoc dependency to flight_testing
341f1b2 is described below

commit 341f1b25c09f1d0225ba2c18c931805d9ecbd2c1
Author: Korn, Uwe <[email protected]>
AuthorDate: Fri Feb 15 10:00:55 2019 -0600

    ARROW-4585: [C++] Add protoc dependency to flight_testing
    
    This leads to the necessary change in `build.ninja`
    ```diff
    --- build.ninja.orig        2019-02-15 16:07:48.000000000 +0100
    +++ build.ninja     2019-02-15 16:10:25.000000000 +0100
    @@ -4863,7 +4863,7 @@
     #############################################
     # Order-only phony target for arrow_flight_testing_objlib
    
    -build cmake_object_order_depends_target_arrow_flight_testing_objlib: phony 
|| src/arrow/flight/CMakeFiles/arrow_flight_testing_objlib.dir
    +build cmake_object_order_depends_target_arrow_flight_testing_objlib: phony 
|| src/arrow/flight/flight_grpc_gen
    ```
    
    Author: Korn, Uwe <[email protected]>
    
    Closes #3658 from xhochy/ARROW-4585 and squashes the following commits:
    
    b71ff9215 <Korn, Uwe> ARROW-4585:  Add protoc dependency to flight_testing
---
 cpp/src/arrow/flight/CMakeLists.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cpp/src/arrow/flight/CMakeLists.txt 
b/cpp/src/arrow/flight/CMakeLists.txt
index a51f4fe..6cab082 100644
--- a/cpp/src/arrow/flight/CMakeLists.txt
+++ b/cpp/src/arrow/flight/CMakeLists.txt
@@ -91,6 +91,7 @@ if(ARROW_BUILD_TESTS OR ARROW_BUILD_BENCHMARKS)
                 test-util.cc
                 DEPENDENCIES
                 ${GTEST_LIBRARY}
+                flight_grpc_gen
                 SHARED_LINK_LIBS
                 arrow_shared
                 arrow_flight_shared

Reply via email to