You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2019/01/25 10:38:07 UTC

[arrow] branch master updated: ARROW-4329: Python should include the parquet headers

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

kszucs 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 32f91b2  ARROW-4329: Python should include the parquet headers
32f91b2 is described below

commit 32f91b2c923809f8413c3a28efe0d0e0545fcc21
Author: Peter <cH...@netflix.com>
AuthorDate: Fri Jan 25 11:37:39 2019 +0100

    ARROW-4329: Python should include the parquet headers
    
    When using PYARROW_BUILD_PARQUET the parquet headers should be included along
    side the arrow and plasma headers.
    
    Author: Peter <cH...@netflix.com>
    
    Closes #3450 from cHYzZQo/python-parquet-headers and squashes the following commits:
    
    ce744c5d <Peter> ARROW-4329: Python should include the parquet and gandiva headers
---
 python/CMakeLists.txt | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt
index 3a048cc..7d2882e 100644
--- a/python/CMakeLists.txt
+++ b/python/CMakeLists.txt
@@ -414,6 +414,10 @@ if (PYARROW_BUILD_PARQUET)
   endif()
   include_directories(SYSTEM ${PARQUET_INCLUDE_DIR})
 
+  if (PYARROW_BUNDLE_ARROW_CPP)
+    file(COPY ${PARQUET_INCLUDE_DIR}/parquet DESTINATION ${BUILD_OUTPUT_ROOT_DIRECTORY}/include)
+  endif()
+
   if (PYARROW_PARQUET_USE_SHARED)
     if (PYARROW_BUNDLE_ARROW_CPP)
       bundle_arrow_lib(PARQUET_SHARED_LIB
@@ -504,6 +508,8 @@ if (PYARROW_BUILD_GANDIVA)
   include_directories(SYSTEM ${GANDIVA_INCLUDE_DIR})
 
   if (PYARROW_BUNDLE_ARROW_CPP)
+    file(COPY ${GANDIVA_INCLUDE_DIR}/gandiva DESTINATION ${BUILD_OUTPUT_ROOT_DIRECTORY}/include)
+
     bundle_arrow_lib(GANDIVA_SHARED_LIB
       ABI_VERSION ${ARROW_ABI_VERSION}
       SO_VERSION ${ARROW_SO_VERSION})