]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
cmake: drop `LDAP_DEPRECATED=1` macro, to sync with autotools
authorViktor Szakats <commit@vsz.me>
Sat, 1 Feb 2025 11:55:47 +0000 (12:55 +0100)
committerViktor Szakats <commit@vsz.me>
Thu, 6 Feb 2025 23:07:38 +0000 (00:07 +0100)
We set this macro to silence a warning inside `openldap.h`. With this
warning now silenced by using `-isystem`, we can drop it. Also it never
had to be set to `1`.

Also enable OpenLDAP in a CMake GHA/macos job.

Follow-up to 445fb81237342ff1ec177a61241a53778570526f #14763
Follow-up to 751e168d93b4a58f3fbbe2908c0041ae2f934329 #12024

Closes #16146

.github/workflows/macos.yml
CMakeLists.txt

index 2681313227ad443390b1a067fa0e885b2146ebb2..dcdede5217ae6bbee514025aad833505225373da 100644 (file)
@@ -147,9 +147,9 @@ jobs:
           - name: 'wolfSSL !ldap brotli zstd'
             install: brotli wolfssl zstd
             generate: -DCURL_USE_WOLFSSL=ON -DCURL_DISABLE_LDAP=ON -DUSE_ECH=ON
-          - name: 'mbedTLS !ldap brotli zstd'
-            install: brotli mbedtls zstd
-            generate: -DCURL_USE_MBEDTLS=ON -DCURL_DISABLE_LDAP=ON
+          - name: 'mbedTLS openldap brotli zstd'
+            install: brotli mbedtls zstd openldap
+            generate: -DCURL_USE_MBEDTLS=ON -DLDAP_INCLUDE_DIR="$(brew --prefix openldap)/include" -DLDAP_LIBRARY="$(brew --prefix openldap)/lib/libldap.dylib" -DLDAP_LBER_LIBRARY="$(brew --prefix openldap)/lib/liblber.dylib"
           - name: 'GnuTLS !ldap krb5'
             install: gnutls nettle krb5
             generate: -DCURL_USE_GNUTLS=ON -DCURL_USE_OPENSSL=OFF -DCURL_USE_GSSAPI=ON -DGSS_ROOT_DIR=$(brew --prefix krb5) -DCURL_DISABLE_LDAP=ON -DUSE_SSLS_EXPORT=ON
index 5f7feecfec153c06633b56a6d663961866f8de4b..810dcbbc6da6dfd88217fd1ed3ad2f75fd233f42 100644 (file)
@@ -1209,7 +1209,6 @@ if(NOT CURL_DISABLE_LDAP)
 
       if(HAVE_LDAP_INIT_FD)
         set(USE_OPENLDAP ON)
-        add_definitions("-DLDAP_DEPRECATED=1")
       endif()
       if(NOT CURL_DISABLE_LDAPS)
         set(HAVE_LDAP_SSL ON)