raulcd commented on code in PR #14097: URL: https://github.com/apache/arrow/pull/14097#discussion_r969595423
########## docs/source/cpp/build_system.rst: ########## @@ -67,11 +67,55 @@ CMake variables: In addition, it will have created some targets that you can link against (note these are plain strings, not variables): -* ``arrow_shared`` links to the Arrow shared libraries -* ``arrow_static`` links to the Arrow static libraries +* ``Arrow::arrow_shared`` links to the Arrow shared libraries +* ``Arrow::arrow_static`` links to the Arrow static libraries + +For backwards compatibility purposes the ``arrow_shared`` and ``arrow_static`` +targets are also available but we recomend using ``Arrow::arrow_shared`` and +``Arrow::arrow_static`` respectively. In most cases, it is recommended to use the Arrow shared libraries. +If Arrow is installed on a custom path instead of a common system one you +will have to add the path where Arrow is installed to ``CMAKE_PREFIX_PATH``. Review Comment: Thanks a lot! I've added a couple of examples, let me know if it makes sense or you think there's something more missing. -- 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]
