lidavidm commented on a change in pull request #11889:
URL: https://github.com/apache/arrow/pull/11889#discussion_r765244471
##########
File path: cpp/src/arrow/ArrowConfig.cmake.in
##########
@@ -87,6 +87,10 @@ if(NOT (TARGET arrow_shared OR TARGET arrow_static))
set_target_properties(
arrow_static PROPERTIES INTERFACE_LINK_LIBRARIES
"${arrow_static_interface_link_libraries};arrow_bundled_dependencies")
+ # Propagate dependencies like OpenSSL
+ set_target_properties(
+ arrow_bundled_dependencies PROPERTIES INTERFACE_LINK_LIBRARIES
+ "${arrow_static_interface_link_libraries}")
Review comment:
Ah, the problem here is the order of libraries to the linker. As
described above, currently, the order is OpenSSL, then
libarrow_bundled_dependencies, so linking will fail because
libarrow_bundled_dependencies itself also depends on OpenSSL. Adding this gets
CMake to repeat OpenSSL after libarrow_bundled_dependencies and then the linker
will find the necessary symbols.
--
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]
