Skip to content

Commit

Permalink
cmake: delete unused NEED_LBER_H, HAVE_LDAP_H
Browse files Browse the repository at this point in the history
Both are used by `./configure` internally, not by curl C code.
CMake adds `NEED_LBER_H` to `CURL_TEST_DEFINES`, which is used by
`curl_internal_test()`, but there is no LDAP-related test made with that
call.

Thus, stop detecting and publishing these from CMake.

`NEED_LBER_H` added in 7320e53.
`HAVE_LDAP_H` added in 4c5307b (initial CMake commit).

Closes curl#14690
  • Loading branch information
vszakats committed Sep 19, 2024
1 parent 9413949 commit 8d32e87
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 36 deletions.
2 changes: 2 additions & 0 deletions .github/scripts/cmp-config.pl
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@
'#define HAVE_GETHOSTBYNAME 1' => 1,
'#define HAVE_INTTYPES_H 1' => 1,
'#define HAVE_IOCTL 1' => 1,
'#define HAVE_LDAP_H 1' => 1,
'#define HAVE_LDAP_SSL 1' => 1,
'#define HAVE_LIBBROTLIDEC 1' => 1,
'#define HAVE_LIBSOCKET 1' => 1,
Expand All @@ -73,6 +74,7 @@
'#define HAVE_ZSTD 1' => 1,
'#define HAVE_ZSTD_H 1' => 1,
'#define LT_OBJDIR ".libs/"' => 1,
'#define NEED_LBER_H 1' => 1,
'#define OS "Linux"' => 1,
'#define OS "x86_64-pc-linux-gnu"' => 1,
'#define PACKAGE "curl"' => 1,
Expand Down
30 changes: 0 additions & 30 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -974,21 +974,6 @@ if(NOT CURL_DISABLE_LDAP)
if(CMAKE_LDAP_INCLUDE_DIR)
include_directories(${CMAKE_LDAP_INCLUDE_DIR})
endif()
set(NEED_LBER_H ON)
unset(_header_list)
if(WIN32)
list(APPEND _header_list "windows.h")
endif()
if(HAVE_SYS_TYPES_H)
list(APPEND _header_list "sys/types.h")
endif()
list(APPEND _header_list "ldap.h")

set(_include_string "")
foreach(_header IN LISTS _header_list)
set(_include_string "${_include_string}#include <${_header}>\n")
endforeach()

list(APPEND CMAKE_REQUIRED_DEFINITIONS "-DLDAP_DEPRECATED=1")
list(APPEND CMAKE_REQUIRED_LIBRARIES ${CMAKE_LDAP_LIB})
set(CURL_LIBS "${CMAKE_LDAP_LIB};${CURL_LIBS}")
Expand All @@ -997,21 +982,6 @@ if(NOT CURL_DISABLE_LDAP)
set(CURL_LIBS "${CMAKE_LBER_LIB};${CURL_LIBS}")
endif()

check_c_source_compiles("
${_include_string}
int main(int argc, char ** argv)
{
BerValue *bvp = NULL;
BerElement *bep = ber_init(bvp);
ber_free(bep, 1);
return 0;
}" NOT_NEED_LBER_H)
if(NOT_NEED_LBER_H)
set(NEED_LBER_H OFF)
else()
set(CURL_TEST_DEFINES "${CURL_TEST_DEFINES} -DNEED_LBER_H")
endif()

check_function_exists("ldap_url_parse" HAVE_LDAP_URL_PARSE)
check_function_exists("ldap_init_fd" HAVE_LDAP_INIT_FD)

Expand Down
6 changes: 0 additions & 6 deletions lib/curl_config.h.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -353,9 +353,6 @@
/* Define to 1 if you have the lber.h header file. */
#cmakedefine HAVE_LBER_H 1

/* Define to 1 if you have the ldap.h header file. */
#cmakedefine HAVE_LDAP_H 1

/* Use LDAPS implementation */
#cmakedefine HAVE_LDAP_SSL 1

Expand Down Expand Up @@ -608,9 +605,6 @@
/* Define this symbol if your OS supports changing the contents of argv */
#cmakedefine HAVE_WRITABLE_ARGV 1

/* Define to 1 if you need the lber.h header file even with ldap.h */
#cmakedefine NEED_LBER_H 1

/* Define to 1 if you need the malloc.h header file even with stdlib.h */
#cmakedefine NEED_MALLOC_H 1

Expand Down

0 comments on commit 8d32e87

Please sign in to comment.