You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@parquet.apache.org by uw...@apache.org on 2018/04/17 17:08:37 UTC

[parquet-cpp] branch master updated: PARQUET-1270: Install executable tools

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

uwe pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-cpp.git


The following commit(s) were added to refs/heads/master by this push:
     new 456801c  PARQUET-1270: Install executable tools
456801c is described below

commit 456801caedcb7f448b21dd1ac3f2c8120e4230cd
Author: Antoine Pitrou <an...@python.org>
AuthorDate: Tue Apr 17 19:08:31 2018 +0200

    PARQUET-1270: Install executable tools
    
    "parquet_reader" and friends should be installed along with the Parquet libraries.
    
    Author: Antoine Pitrou <an...@python.org>
    
    Closes #455 from pitrou/PARQUET-1270-install-tools and squashes the following commits:
    
    44c486c [Antoine Pitrou] PARQUET-1270: Install executable tools
---
 tools/CMakeLists.txt | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/tools/CMakeLists.txt b/tools/CMakeLists.txt
index d473d08..0705c83 100644
--- a/tools/CMakeLists.txt
+++ b/tools/CMakeLists.txt
@@ -16,12 +16,16 @@
 # under the License.
 
 if (PARQUET_BUILD_EXECUTABLES)
-  add_executable(parquet-dump-schema parquet-dump-schema.cc)
-  target_link_libraries(parquet-dump-schema parquet_static)
+  set(EXECUTABLE_TOOLS
+    parquet-dump-schema
+    parquet_reader
+    parquet-scan)
 
-  add_executable(parquet_reader parquet_reader.cc)
-  target_link_libraries(parquet_reader parquet_static)
-
-  add_executable(parquet-scan parquet-scan.cc)
-  target_link_libraries(parquet-scan parquet_static)
+  foreach(TOOL ${EXECUTABLE_TOOLS})
+    add_executable(${TOOL} "${TOOL}.cc")
+    target_link_libraries(${TOOL} parquet_static)
+    # Avoid unsetting RPATH when installing
+    set_target_properties(${TOOL} PROPERTIES INSTALL_RPATH_USE_LINK_PATH TRUE)
+    install(TARGETS ${TOOL} RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
+  endforeach(TOOL)
 endif()

-- 
To stop receiving notification emails like this one, please contact
uwe@apache.org.