raulcd commented on code in PR #50650:
URL: https://github.com/apache/arrow/pull/50650#discussion_r3703847985
##########
cpp/cmake_modules/ThirdpartyToolchain.cmake:
##########
@@ -4326,14 +4302,18 @@ function(build_azure_sdk)
set(AZURE_SDK_VENDORED
TRUE
PARENT_SCOPE)
+ list(PREPEND
Review Comment:
Thanks for the explanation. In my opinion using `PREPEND` makes sense here
even if not strictly necessary. The benefit of not having to check all Abseil
CMake targets is enough to justify. Maintaining reverse order based on our
build dependency resolution (build_abseil, build_protobuf: Protobud -> Abseil)
even though not being required makes sense to me.
--
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]
