From: Matt Benjamin <mbenjamin@redhat.com>
Date: Mon, 14 Mar 2016 18:08:53 +0000 (-0400)
Subject: cmake: avoid false-positive LDAP header detect
X-Git-Tag: v10.1.0~107^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F8100%2Fhead;p=ceph.git

cmake: avoid false-positive LDAP header detect

Pre-setting a value in OPENLDAP_INCLUDE_DIR broke the subsequent
test.

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
---

diff --git a/cmake/modules/FindOpenLdap.cmake b/cmake/modules/FindOpenLdap.cmake
index e9b956daff75..42b3edd01d61 100644
--- a/cmake/modules/FindOpenLdap.cmake
+++ b/cmake/modules/FindOpenLdap.cmake
@@ -3,13 +3,13 @@
 # OPENLDAP_PREFIX - where to find ldap.h and libraries
 # OPENLDAP_FOUND - True if found.
 
-set(OPENLDAP_INCLUDE_DIR "${OPENLDAP_PREFIX}/include")
 set(OPENLDAP_LIB_DIR "${OPENLDAP_PREFIX}/lib")
 
 find_path(OPENLDAP_INCLUDE_DIR ldap.h NO_DEFAULT_PATH PATHS
   /usr/include
   /opt/local/include
   /usr/local/include
+  "${OPENLDAP_PREFIX}/include"
   )
 
 find_library(LIBLDAP NAMES ldap)
@@ -23,12 +23,13 @@ endif (OPENLDAP_INCLUDE_DIR AND LIBLDAP AND LIBLBER)
 
 if (OPENLDAP_FOUND)
   message(STATUS "Found ldap: ${OPENLDAP_INCLUDE_DIR}")
-else ()
-  message(STATUS "Failed to find ldap.h")
-  if (OPENLDAP_FIND_REQUIRED)
-    message(FATAL_ERROR "Missing required ldap.h")
-  endif ()
-endif ()
+else (OPENLDAP_FOUND)
+  if (NOT OPENLDAP_INCLUDE_DIR)
+    message(FATAL_ERROR "Missing required ldap.h (openldap-devel)")
+  else (NOT OPENLDAP_INCLUDE_DIR)
+    message (FATAL_ERROR "Missing required LDAP libraries (openldap)")
+  endif (NOT OPENLDAP_INCLUDE_DIR)
+endif (OPENLDAP_FOUND)
 
 set(OPENLDAP_LIBS ${LIBLDAP} ${LIBLBER})