This is an automated email from the ASF dual-hosted git repository.
wesm 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 d143a42 ARROW-4891: [C++] Add zlib headers to include directories
d143a42 is described below
commit d143a42708cd2a64ed79f55fee5fad981b510c37
Author: Korn, Uwe <[email protected]>
AuthorDate: Fri Mar 15 12:23:09 2019 -0500
ARROW-4891: [C++] Add zlib headers to include directories
Author: Korn, Uwe <[email protected]>
Closes #3914 from xhochy/ARROW-4891 and squashes the following commits:
74117a4c2 <Korn, Uwe> ARROW-4891: Add zlib headers to include directories
---
cpp/cmake_modules/ThirdpartyToolchain.cmake | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/cpp/cmake_modules/ThirdpartyToolchain.cmake
b/cpp/cmake_modules/ThirdpartyToolchain.cmake
index 1c290b2..09ee367 100644
--- a/cpp/cmake_modules/ThirdpartyToolchain.cmake
+++ b/cpp/cmake_modules/ThirdpartyToolchain.cmake
@@ -1388,6 +1388,10 @@ endmacro()
if(ARROW_WITH_ZLIB)
resolve_dependency(ZLIB)
+
+ # TODO: Don't use global includes but rather target_include_directories
+ get_target_property(ZLIB_INCLUDE_DIR ZLIB::ZLIB
INTERFACE_INCLUDE_DIRECTORIES)
+ include_directories(SYSTEM ${ZLIB_INCLUDE_DIR})
endif()
macro(build_lz4)
