orc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jcrist <...@git.apache.org>
Subject [GitHub] orc pull request #192: ORC-266: [C++] Cleanup cmake scripts
Date Wed, 22 Nov 2017 21:53:24 GMT
Github user jcrist commented on a diff in the pull request:

    https://github.com/apache/orc/pull/192#discussion_r152686927
  
    --- Diff: cmake_modules/ThirdpartyToolchain.cmake ---
    @@ -41,32 +68,35 @@ if (NOT SNAPPY_FOUND)
         LOG_BUILD 1
         LOG_INSTALL 1
         BUILD_BYPRODUCTS "${SNAPPY_STATIC_LIB}")
    +
    +  set (SNAPPY_VENDORED TRUE)
     endif ()
    -include_directories (SYSTEM ${SNAPPY_INCLUDE_DIRS})
    +
    +include_directories (SYSTEM ${SNAPPY_INCLUDE_DIR})
     add_library (snappy STATIC IMPORTED)
     set_target_properties (snappy PROPERTIES IMPORTED_LOCATION ${SNAPPY_STATIC_LIB})
    -set (SNAPPY_LIBRARIES snappy)
    -add_dependencies (snappy snappy_ep)
    -install(DIRECTORY ${SNAPPY_PREFIX}/lib DESTINATION .
    --- End diff --
    
    Done. Went with `INSTALL_VENDORED_LIBS`, which defaults to on.


---

Mime
View raw message