You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2017/07/26 16:06:27 UTC

arrow git commit: ARROW-1275: [C++] Deafult Snappy static lib suffix updated to "_static"

Repository: arrow
Updated Branches:
  refs/heads/master 5708cd103 -> dca5d96c7


ARROW-1275: [C++] Deafult Snappy static lib suffix updated to "_static"

Author: Max Risuhin <ri...@gmail.com>

Closes #894 from MaxRis/ARROW-1275 and squashes the following commits:

3dcf216 [Max Risuhin] ARROW-1275: [C++] Deafult Snappy static lib suffix updated to "_static"


Project: http://git-wip-us.apache.org/repos/asf/arrow/repo
Commit: http://git-wip-us.apache.org/repos/asf/arrow/commit/dca5d96c
Tree: http://git-wip-us.apache.org/repos/asf/arrow/tree/dca5d96c
Diff: http://git-wip-us.apache.org/repos/asf/arrow/diff/dca5d96c

Branch: refs/heads/master
Commit: dca5d96c7a029c079183e2903db425e486e2deb9
Parents: 5708cd1
Author: Max Risuhin <ri...@gmail.com>
Authored: Wed Jul 26 12:06:21 2017 -0400
Committer: Wes McKinney <we...@twosigma.com>
Committed: Wed Jul 26 12:06:21 2017 -0400

----------------------------------------------------------------------
 cpp/CMakeLists.txt                          | 2 +-
 cpp/cmake_modules/SnappyCMakeLists.txt      | 6 +++---
 cpp/cmake_modules/ThirdpartyToolchain.cmake | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/arrow/blob/dca5d96c/cpp/CMakeLists.txt
----------------------------------------------------------------------
diff --git a/cpp/CMakeLists.txt b/cpp/CMakeLists.txt
index 41af1e1..1e9aef0 100644
--- a/cpp/CMakeLists.txt
+++ b/cpp/CMakeLists.txt
@@ -165,7 +165,7 @@ if("${CMAKE_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_SOURCE_DIR}")
   if (MSVC)
     set(BROTLI_MSVC_STATIC_LIB_SUFFIX "_static" CACHE STRING
       "Brotli static lib suffix used on Windows with MSVC (default _static)")
-    set(SNAPPY_MSVC_STATIC_LIB_SUFFIX "" CACHE STRING
+    set(SNAPPY_MSVC_STATIC_LIB_SUFFIX "_static" CACHE STRING
       "Snappy static lib suffix used on Windows with MSVC (default is empty string)")
     set(ZLIB_MSVC_STATIC_LIB_SUFFIX "libstatic" CACHE STRING
       "Zlib static lib suffix used on Windows with MSVC (default libstatic)")

http://git-wip-us.apache.org/repos/asf/arrow/blob/dca5d96c/cpp/cmake_modules/SnappyCMakeLists.txt
----------------------------------------------------------------------
diff --git a/cpp/cmake_modules/SnappyCMakeLists.txt b/cpp/cmake_modules/SnappyCMakeLists.txt
index 9d0a166..50083ce 100644
--- a/cpp/cmake_modules/SnappyCMakeLists.txt
+++ b/cpp/cmake_modules/SnappyCMakeLists.txt
@@ -68,10 +68,10 @@ set(SNAPPY_SRCS snappy.cc
   snappy-stubs-public.h)
 
 add_library(snappy SHARED ${SNAPPY_SRCS})
-add_library(snappystatic STATIC ${SNAPPY_SRCS})
+add_library(snappy_static STATIC ${SNAPPY_SRCS})
 
 TARGET_COMPILE_DEFINITIONS(snappy PRIVATE -DHAVE_CONFIG_H)
-TARGET_COMPILE_DEFINITIONS(snappystatic PRIVATE -DHAVE_CONFIG_H)
+TARGET_COMPILE_DEFINITIONS(snappy_static PRIVATE -DHAVE_CONFIG_H)
 
 install(FILES snappy.h
   snappy-c.h
@@ -79,7 +79,7 @@ install(FILES snappy.h
   ${snappy_BINARY_DIR}/snappy-stubs-public.h
   DESTINATION include)
 
-install(TARGETS snappy snappystatic
+install(TARGETS snappy snappy_static
   RUNTIME DESTINATION bin
   LIBRARY DESTINATION lib
   ARCHIVE DESTINATION lib)

http://git-wip-us.apache.org/repos/asf/arrow/blob/dca5d96c/cpp/cmake_modules/ThirdpartyToolchain.cmake
----------------------------------------------------------------------
diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index b9d9823..721e866 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -501,7 +501,7 @@ if (ARROW_WITH_SNAPPY)
     set(SNAPPY_HOME "${SNAPPY_PREFIX}")
     set(SNAPPY_INCLUDE_DIR "${SNAPPY_PREFIX}/include")
     if (MSVC)
-      set(SNAPPY_STATIC_LIB_NAME snappystatic)
+      set(SNAPPY_STATIC_LIB_NAME snappy_static)
     else()
       set(SNAPPY_STATIC_LIB_NAME snappy)
     endif()