commit c444fb581d2157c260f36db091a7cdd4511c6e32
author Nobuaki Sukegawa <nsukeg@gmail.com> Fri Jan 02 23:16:55 2015 +0900
committer Roger Meier <roger@apache.org> Sat Jan 03 10:08:09 2015 +0100
tree 2cd9f0a1af9b77bf82d24e0330bb91202ebfa730
parent 5c10fd8cb146181e1c835787167812fcf1cbf169

THRIFT-2925 CMake build does not work with OpenSSL nor anything installed in non-system location

This diff fixes them by correcting invalid CMake variable names:
* OPENSSL_... rather than OpenSSL_...
* LIBEVENT_... rather than Libevent_...
* Boost_INCLUDE_DIRS rather than Boost_INCLUDE_DIR
* LIBEVENT_INCLUDE_DIRS rather than LIBEVENT_INCLUDE_DIR
* ZLIB_INCLUDE_DIRS rather than ZLIB_INCLUDE_DIR

Note:
* OPENSSL_INCLUDE_DIR is correct (rather than ..._DIRS)
* Boost_INCLUDE_DIR exists and actually works for most cases but
  Boost_INCLUDE_DIRS is the one desinged to be included

Also, library headers are now included as SYSTEM headers.

cmake/DefineOptions.cmake

diff --git a/cmake/DefineOptions.cmake b/cmake/DefineOptions.cmake
index d7144ea..391247e 100644
--- a/cmake/DefineOptions.cmake
+++ b/cmake/DefineOptions.cmake
@@ -59,7 +59,7 @@
 endif()
 find_package(OpenSSL QUIET)
 CMAKE_DEPENDENT_OPTION(WITH_OPENSSL "Build with OpenSSL support" ON
-                       "OpenSSL_FOUND" OFF)
+                       "OPENSSL_FOUND" OFF)
 option(WITH_BOOSTTHREADS "Build with Boost thread support" OFF)
 option(WITH_STDTHREADS "Build with C++ std::thread support" OFF)