cmake .. will not activate ENABLE_AWS_S3 by default. (#922)

Author: yokofly

contrib/aws-cmake/CMakeLists.txt

@@ -2,7 +2,7 @@ set(ENABLE_AWS_S3_DEFAULT OFF)
 set(ENABLE_AWS_MSK_IAM_DEFAULT OFF) # proton: added
 
 if(ENABLE_LIBRARIES AND (OS_LINUX OR OS_DARWIN) AND TARGET OpenSSL::Crypto)
-    set(ENABLE_AWS_S3_DEFAULT ON)
+    set(ENABLE_AWS_S3_DEFAULT OFF) # proton: added https://github.com/timeplus-io/proton/issues/918
     set(ENABLE_AWS_MSK_IAM_DEFAULT ON) # proton: added
 endif()
 
@@ -17,6 +17,11 @@ if(ENABLE_AWS_S3 OR ENABLE_AWS_MSK_IAM) # proton: updated
     endif()
 endif()
 
+if(NOT ENABLE_AWS_S3)
+    message(STATUS "Not using AWS S3")
+    return()
+endif()
+
 if(NOT (ENABLE_AWS_S3 OR ENABLE_AWS_MSK_IAM)) # proton: updated
     message(STATUS "Not using AWS SDK") # proton: updated
     return()

src/configure_config.cmake

@@ -62,7 +62,7 @@ if (TARGET ch_contrib::datasketches)
     set(USE_DATASKETCHES 1)
 endif()
 if (TARGET ch_contrib::aws_s3)
-    set(USE_AWS_S3 0)
+    set(USE_AWS_S3 1)
 endif()
 # proton: starts
 if (TARGET ch_contrib::aws_msk_iam)