You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@orc.apache.org by jcrist <gi...@git.apache.org> on 2017/11/22 21:53:24 UTC

[GitHub] orc pull request #192: ORC-266: [C++] Cleanup cmake scripts

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.


---