From acec11d515d70db1b35a9b7c6fb2239a8e5b2eb7 Mon Sep 17 00:00:00 2001
From: Ward Fisher <wfisher@ucar.edu>
Date: Wed, 3 Apr 2024 11:42:59 -0600
Subject: [PATCH] Use modern cmake nomenclature for curl.

---
 cmake/dependencies.cmake | 30 ++++++++++++++----------------
 libdap2/CMakeLists.txt   |  2 +-
 libdap4/CMakeLists.txt   |  2 +-
 liblib/CMakeLists.txt    |  6 +++++-
 libsrc/CMakeLists.txt    |  2 ++
 oc2/CMakeLists.txt       |  2 +-
 6 files changed, 24 insertions(+), 20 deletions(-)

diff --git a/cmake/dependencies.cmake b/cmake/dependencies.cmake
index d74a6ecd5..2b5905f5e 100644
--- a/cmake/dependencies.cmake
+++ b/cmake/dependencies.cmake
@@ -218,25 +218,23 @@ endif(USE_HDF5)
 ################################
 # See if we have libcurl
 find_package(CURL)
-target_compile_options(netcdf
-  PRIVATE
-    -DCURL_STATICLIB=1
-)
-target_include_directories(netcdf
-  PRIVATE
-    ${CURL_INCLUDE_DIRS}
-)
-
-
-
-MESSAGE(STATUS "Found CURL_INCLUDE_DIRS: ${CURL_INCLUDE_DIRS}")
-# Define a test flag for have curl library
-if(CURL_LIBRARIES OR CURL_LIBRARY)
+#target_compile_options(netcdf
+#  PRIVATE
+#    -DCURL_STATICLIB=1
+#)
+#target_include_directories(netcdf
+#  PRIVATE
+#    ${CURL_INCLUDE_DIRS}
+#)
+if(CURL_FOUND)
   set(FOUND_CURL TRUE)
+  target_link_libraries(netcdf
+  PRIVATE
+  CURL::libcurl
+)
 else()
   set(FOUND_CURL FALSE)
-endif()
-set(FOUND_CURL ${FOUND_CURL} TRUE )
+endif(CURL_FOUND)
 
 # Start disabling if curl not found
 if(NOT FOUND_CURL)
diff --git a/libdap2/CMakeLists.txt b/libdap2/CMakeLists.txt
index ed59fc1cd..9e09abe07 100644
--- a/libdap2/CMakeLists.txt
+++ b/libdap2/CMakeLists.txt
@@ -27,7 +27,7 @@ set_property(SOURCE ncd2dispatch.c
 add_library(dap2 OBJECT ${dap2_SOURCES})
 
 
-target_link_libraries(dap2 PUBLIC ${CURL_LIBRARIES})
+target_link_libraries(dap2 PUBLIC CURL::libcurl ${CURL_LIBRARIES})
 target_include_directories(dap2 PUBLIC ${CURL_INCLUDE_DIRS})
 target_compile_options(dap2
   PRIVATE
diff --git a/libdap4/CMakeLists.txt b/libdap4/CMakeLists.txt
index 4b21a642b..e410977c1 100644
--- a/libdap4/CMakeLists.txt
+++ b/libdap4/CMakeLists.txt
@@ -20,7 +20,7 @@ set_property(SOURCE d4meta.c
     SKIP_UNITY_BUILD_INCLUSION ON)
 
 add_library(dap4 OBJECT ${dap4_SOURCES})
-target_link_libraries(dap4 PUBLIC ${CURL_LIBRARIES})
+target_link_libraries(dap4 PUBLIC CURL::libcurl ${CURL_LIBRARIES})
 target_include_directories(dap4 PUBLIC ${CURL_INCLUDE_DIRS})
 target_compile_options(dap4
   PRIVATE
diff --git a/liblib/CMakeLists.txt b/liblib/CMakeLists.txt
index 36a709f65..e0fe9f5c9 100644
--- a/liblib/CMakeLists.txt
+++ b/liblib/CMakeLists.txt
@@ -42,6 +42,10 @@ if(USE_HDF4)
   )
 endif()
 
+if(FOUND_CURL)
+  target_link_libraries(netcdf PRIVATE CURL::libcurl)
+endif()
+
 if(NETCDF_ENABLE_DAP2)
   target_sources(netcdf
     PRIVATE
@@ -144,7 +148,7 @@ if(USE_HDF5)
 endif()
 
 if(FOUND_CURL)
-  set(TLL_LIBS ${TLL_LIBS} ${CURL_LIBRARIES})
+  set(TLL_LIBS ${TLL_LIBS} CURL::libcurl ${CURL_LIBRARIES})
 endif()
 
 if(USE_HDF4)
diff --git a/libsrc/CMakeLists.txt b/libsrc/CMakeLists.txt
index a3ea899c9..022c6f42c 100644
--- a/libsrc/CMakeLists.txt
+++ b/libsrc/CMakeLists.txt
@@ -42,6 +42,7 @@ ENDif (USE_FFIO)
 
 if (NETCDF_ENABLE_BYTERANGE)
   list(APPEND libsrc_SOURCES httpio.c)
+
 if (NETCDF_ENABLE_S3)
   list(APPEND libsrc_SOURCES s3io.c)
 endif(NETCDF_ENABLE_S3)
@@ -55,6 +56,7 @@ endif()
 
 if (NETCDF_ENABLE_BYTERANGE)
   target_include_directories(netcdf3 PUBLIC ${CURL_INCLUDE_DIRS})
+  target_link_libraries(netcdf3 PUBLIC CURL::libcurl)
   target_compile_options(netcdf3
     PRIVATE
       -DCURL_STATICLIB=1
diff --git a/oc2/CMakeLists.txt b/oc2/CMakeLists.txt
index 36a675c76..01b2e998e 100644
--- a/oc2/CMakeLists.txt
+++ b/oc2/CMakeLists.txt
@@ -15,7 +15,7 @@ endif()
 if(STATUS_PARALLEL)
   target_link_libraries(oc2 PUBLIC MPI::MPI_C)
 endif(STATUS_PARALLEL)
-target_link_libraries(oc2 PUBLIC ${CURL_LIBRARIES})
+target_link_libraries(oc2 PUBLIC CURL::libcurl ${CURL_LIBRARIES})
 target_include_directories(oc2 PUBLIC ${CURL_INCLUDE_DIRS})
 target_compile_options(oc2
   PRIVATE