From 409f5521dd1e344859aebaeb4bf6b0a8c239b88b Mon Sep 17 00:00:00 2001
From: Seth Shelnutt <Shelnutt2@gmail.com>
Date: Tue, 20 Jun 2023 07:07:38 -0400
Subject: [PATCH] Minimize object library linking for s3

---
 tiledb/sm/filesystem/CMakeLists.txt    | 38 --------------------------
 tiledb/sm/filesystem/s3/CMakeLists.txt |  6 ++--
 2 files changed, 3 insertions(+), 41 deletions(-)

diff --git a/tiledb/sm/filesystem/CMakeLists.txt b/tiledb/sm/filesystem/CMakeLists.txt
index 5bdb4850604e..395edb9f1e6a 100644
--- a/tiledb/sm/filesystem/CMakeLists.txt
+++ b/tiledb/sm/filesystem/CMakeLists.txt
@@ -52,26 +52,6 @@ commence(object_library vfs)
             s3.cc
             s3_thread_pool_executor.cc
         )
-        if(WIN32)
-            if(MSVC)
-                find_library(BCRYPT_LIBRARY bcrypt)
-                message(STATUS "Found Win32 lib bcrypt: ${BCRYPT_LIBRARY}")
-                this_target_link_libraries(${BCRYPT_LIBRARY})
-            else()
-                message(STATUS "Linking to Win32 lib bcrypt")
-                this_target_link_libraries(-lbcrypt)
-            endif()
-        else()
-            find_package(OpenSSL_EP REQUIRED)
-            this_target_link_libraries(INTERFACE OpenSSL::Crypto OpenSSL::SSL)
-        endif()
-        if (NOT WIN32)
-            find_package(Curl_EP REQUIRED)
-            target_link_libraries(s3_providers
-                    INTERFACE
-                    CURL::libcurl
-                    )
-        endif()
     endif()
     this_target_sources(${VFS_SOURCES})
     this_target_object_libraries(
@@ -100,24 +80,6 @@ commence(object_library vfs)
             this_target_link_libraries(-lshlwapi)
         endif()
     endif()
-    # Add S3 dependencies
-    if (TILEDB_S3)
-        if (TILEDB_VCPKG)
-            find_package(AWSSDK_EP REQUIRED COMPONENTS s3)
-            target_link_libraries(vfs INTERFACE ${AWSSDK_LINK_LIBRARIES})
-        else()
-            find_package(AWSSDK_EP REQUIRED COMPONENTS s3)
-            target_link_libraries(vfs INTERFACE
-                AWSSDK::aws-cpp-sdk-s3
-                AWSSDK::aws-cpp-sdk-core
-                AWSSDK::aws-c-event-stream
-                AWSSDK::aws-checksums
-                AWSSDK::aws-c-common
-                AWSSDK::aws-cpp-sdk-identity-management
-                AWSSDK::aws-cpp-sdk-sts
-            )
-        endif()
-    endif()
 conclude(object_library)
 
 add_test_subdirectory()
diff --git a/tiledb/sm/filesystem/s3/CMakeLists.txt b/tiledb/sm/filesystem/s3/CMakeLists.txt
index 8e782aa6299b..0395469f73dc 100644
--- a/tiledb/sm/filesystem/s3/CMakeLists.txt
+++ b/tiledb/sm/filesystem/s3/CMakeLists.txt
@@ -80,10 +80,10 @@ commence(object_library s3_providers)
         else()
             find_package(OpenSSL_EP REQUIRED)
             this_target_link_libraries(INTERFACE OpenSSL::Crypto OpenSSL::SSL)
-        endif()
-        if (NOT WIN32)
+
+            # Add curl
             find_package(Curl_EP REQUIRED)
-            target_link_libraries(s3_providers
+            this_target_link_libraries(
                     INTERFACE
                     CURL::libcurl
             )