Merge pull request #45 from osalloum/bugfix/sts-build

STS needs to be statically linked
duckdb · Jun 12, 2024 · 7b5d5a5 · 7b5d5a5
2 parents 9201a9b + f460a86
commit 7b5d5a5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -16,7 +16,7 @@ build_loadable_extension(${TARGET_NAME} ${PARAMETERS} ${EXTENSION_SOURCES})
 # Weirdly we need to manually to this, otherwise linking against
 # ${AWSSDK_LINK_LIBRARIES} fails for some reason
 find_package(ZLIB REQUIRED)
-find_package(AWSSDK REQUIRED COMPONENTS core)
+find_package(AWSSDK REQUIRED COMPONENTS core sts)
 
 # Build static lib
 target_include_directories(${EXTENSION_NAME}

diff --git a/vcpkg.json b/vcpkg.json
@@ -1,9 +1,12 @@
 {
   "dependencies": [
     "zlib",
-    "aws-sdk-cpp",
+    {
+      "name": "aws-sdk-cpp",
+      "features": [ "sts" ]
+    },
     "openssl"
   ],
   "builtin-baseline": "a1a1cbc975abf909a6c8985a6a2b8fe20bbd9bd6",
   "overrides": [{"name": "openssl", "version": "3.0.8"}]
-}
+}