[ARROW-16168] [C++][CMake] Use target instead of include_directories(SYSTEM) to add include path

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/20188
• Assignee: Kouhei Sutou
• Reporter: Kouhei Sutou
• Created: 12/Apr/22 06:26
• Updated: 11/Jan/23 11:42
• Resolved: 11/May/22 20:24

Description

There are many "# TODO: Don't use global includes but rather target_include_directories" comments in cpp/cmake_modules/ThirdpartyToolchain.cmake.

Issue Links

• relates to: ARROW-16089 [Packaging] Add support for Coan C/C++ package manager
• links to: GitHub Pull Request #12861