js8544 commented on code in PR #33808:
URL: https://github.com/apache/arrow/pull/33808#discussion_r1090193253
##########
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##########
@@ -4673,7 +4761,19 @@ macro(build_awssdk)
aws-cpp-sdk-core
aws-c-event-stream
aws-checksums
- aws-c-common)
+ aws-c-common
+ aws-c-auth
+ aws-c-cal
+ aws-c-compression
+ aws-c-http
+ aws-c-io
+ aws-c-mqtt
+ aws-c-s3
+ aws-c-sdkutils
+ s2n
+ crypto
+ ssl
Review Comment:
It turns out they are not needed. I've removed these two.
##########
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##########
@@ -4714,35 +4814,145 @@ macro(build_awssdk)
DEPENDS aws_c_common_ep)
add_dependencies(AWS::aws-checksums aws_checksums_ep)
+ externalproject_add(s2n_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${S2N_SOURCE_URL}
+ URL_HASH "SHA256=${ARROW_S2N_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${S2N_STATIC_LIBRARY})
+ add_dependencies(AWS::s2n s2n_ep)
+
+ externalproject_add(aws_c_cal_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_CAL_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_CAL_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_CAL_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep)
+ add_dependencies(AWS::aws-c-cal aws_c_cal_ep)
+
+ externalproject_add(aws_c_io_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_IO_SOURCE_URL}
+ URL_HASH "SHA256=${ARROW_AWS_C_IO_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_IO_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep s2n_ep aws_c_cal_ep)
+ add_dependencies(AWS::aws-c-io aws_c_io_ep)
+
externalproject_add(aws_c_event_stream_ep
${EP_COMMON_OPTIONS}
URL ${AWS_C_EVENT_STREAM_SOURCE_URL}
URL_HASH
"SHA256=${ARROW_AWS_C_EVENT_STREAM_BUILD_SHA256_CHECKSUM}"
CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
BUILD_BYPRODUCTS ${AWS_C_EVENT_STREAM_STATIC_LIBRARY}
- DEPENDS aws_checksums_ep)
+ DEPENDS aws_checksums_ep s2n_ep aws_c_io_ep)
add_dependencies(AWS::aws-c-event-stream aws_c_event_stream_ep)
- set(AWSSDK_PATCH_COMMAND)
- if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU" AND CMAKE_CXX_COMPILER_VERSION
VERSION_GREATER
- "10")
- # Workaround for https://github.com/aws/aws-sdk-cpp/issues/1750
- set(AWSSDK_PATCH_COMMAND "sed" "-i.bak" "-e" "s/\"-Werror\"//g"
- "<SOURCE_DIR>/cmake/compiler_settings.cmake")
- endif()
+ externalproject_add(aws_c_sdkutils_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_SDKUTILS_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_SDKUTILS_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_SDKUTILS_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep)
+ add_dependencies(AWS::aws-c-sdkutils aws_c_sdkutils_ep)
+
+ externalproject_add(aws_c_compression_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_COMPRESSION_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_COMPRESSION_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_COMPRESSION_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep)
+ add_dependencies(AWS::aws-c-compression aws_c_compression_ep)
+
+ externalproject_add(aws_c_http_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_HTTP_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_HTTP_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_HTTP_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep s2n_ep aws_c_io_ep)
+ add_dependencies(AWS::aws-c-http aws_c_http_ep)
+
+ externalproject_add(aws_c_mqtt_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_MQTT_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_MQTT_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_MQTT_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep s2n_ep aws_c_io_ep aws_c_http_ep)
+ add_dependencies(AWS::aws-c-mqtt aws_c_mqtt_ep)
+
+ externalproject_add(aws_c_auth_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_AUTH_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_C_AUTH_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_AUTH_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep
+ aws_c_sdkutils_ep
+ s2n_ep
+ aws_c_io_ep
+ aws_c_http_ep)
+ add_dependencies(AWS::aws-c-auth aws_c_auth_ep)
+
+ externalproject_add(aws_c_s3_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_C_S3_SOURCE_URL}
+ URL_HASH "SHA256=${ARROW_AWS_C_S3_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_C_S3_STATIC_LIBRARY}
+ DEPENDS aws_c_common_ep
+ aws_c_cal_ep
+ s2n_ep
+ aws_c_io_ep
+ aws_c_http_ep
+ aws_c_auth_ep)
+ add_dependencies(AWS::aws-c-s3 aws_c_s3_ep)
+
+ externalproject_add(aws_lc_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_LC_SOURCE_URL}
+ URL_HASH "SHA256=${ARROW_AWS_LC_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_COMMON_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${CRYPTO_STATIC_LIBRARY}
${SSL_STATIC_LIBRARY})
+ add_dependencies(AWS::crypto aws_lc_ep)
+ add_dependencies(AWS::ssl aws_lc_ep)
+
+ externalproject_add(aws_crt_cpp_ep
+ ${EP_COMMON_OPTIONS}
+ URL ${AWS_CRT_CPP_SOURCE_URL}
+ URL_HASH
"SHA256=${ARROW_AWS_CRT_CPP_BUILD_SHA256_CHECKSUM}"
+ CMAKE_ARGS ${AWSSDK_CMAKE_ARGS}
+ BUILD_BYPRODUCTS ${AWS_CRT_CPP_STATIC_LIBRARY}
+ DEPENDS aws_c_auth_ep
+ aws_c_cal_ep
+ aws_c_common_ep
+ aws_c_compression_ep
+ aws_c_event_stream_ep
+ aws_c_http_ep
+ aws_c_io_ep
+ aws_c_mqtt_ep
+ aws_c_s3_ep
+ aws_c_sdkutils_ep
+ aws_checksums_ep
+ aws_lc_ep
+ s2n_ep)
+ add_dependencies(AWS::aws-crt-cpp aws_crt_cpp_ep)
externalproject_add(awssdk_ep
${EP_COMMON_OPTIONS}
URL ${AWSSDK_SOURCE_URL}
URL_HASH "SHA256=${ARROW_AWSSDK_BUILD_SHA256_CHECKSUM}"
CMAKE_ARGS ${AWSSDK_CMAKE_ARGS}
- PATCH_COMMAND ${AWSSDK_PATCH_COMMAND}
BUILD_BYPRODUCTS
${AWS_CPP_SDK_COGNITO_IDENTITY_STATIC_LIBRARY}
${AWS_CPP_SDK_CORE_STATIC_LIBRARY}
${AWS_CPP_SDK_IDENTITY_MANAGEMENT_STATIC_LIBRARY}
${AWS_CPP_SDK_S3_STATIC_LIBRARY}
${AWS_CPP_SDK_STS_STATIC_LIBRARY}
- DEPENDS aws_c_event_stream_ep)
+ DEPENDS aws_c_event_stream_ep aws_crt_cpp_ep)
Review Comment:
done
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
