nateprewitt commented on code in PR #48971:
URL: https://github.com/apache/arrow/pull/48971#discussion_r2729212243
##########
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##########
@@ -4053,6 +4061,27 @@ endif()
function(build_azure_sdk)
message(STATUS "Building Azure SDK for C++ from source")
+
+ # On Windows, Azure SDK's WinHTTP transport requires WIL (Windows
Implementation Libraries).
+ # Fetch WIL before Azure SDK so the WIL::WIL target is available.
+ if(WIN32)
+ message(STATUS "Fetching WIL (Windows Implementation Libraries) for Azure
SDK")
+ fetchcontent_declare(wil
+ ${FC_DECLARE_COMMON_OPTIONS}
+ URL ${ARROW_WIL_URL}
+ URL_HASH "SHA256=${ARROW_WIL_BUILD_SHA256_CHECKSUM}")
+ set(WIL_BUILD_PACKAGING OFF)
+ set(WIL_BUILD_TESTS OFF)
+ fetchcontent_makeavailable(wil)
+ # Create a minimal config file so Azure SDK's find_package(wil CONFIG)
succeeds.
+ # The WIL::WIL target already exists from FetchContent above.
+ file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/wil-config/wilConfig.cmake"
+ "# WIL loaded via FetchContent - target WIL::WIL already exists\n")
+ set(wil_DIR
+ "${CMAKE_CURRENT_BINARY_DIR}/wil-config"
+ CACHE PATH "" FORCE)
Review Comment:
Should be updated, thanks for the pointer!
--
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]
