KHARSHAVARDHAN-eng commented on issue #50753: URL: https://github.com/apache/arrow/issues/50753#issuecomment-5152301511
Implemented the change by extending `resolve_dependency()` with `STATIC_INSTALL_INTERFACE_LIBS`, so static interface dependencies can now be declared alongside dependency resolution instead of being maintained separately. I also migrated the existing applicable dependencies to this mechanism and removed the corresponding duplicate declarations from the Arrow and Flight CMakeLists. For verification, I tested both SYSTEM and BUNDLED static configurations and inspected the generated `ArrowTargets.cmake` to confirm the expected interface dependencies are propagated correctly without leaking SYSTEM targets into BUNDLED builds. I hope this covers the intended scope of #50753. If there are any changes, additional tests, or adjustments you'd prefer, I'm happy to address them. Thanks! -- 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]
