[ARROW-17394] [C++][Parquet] Fix parquet_static dependencies

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 10.0.0
• Component/s: C++, Parquet
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/32665
• Assignee: Kouhei Sutou
• Reporter: Kouhei Sutou
• Created: 12/Aug/22 02:11
• Updated: 11/Jan/23 11:50
• Resolved: 15/Aug/22 06:47

Description

See also: https://github.com/microsoft/vcpkg/issues/22552#issuecomment-1211341648

I tried #22552 (comment) (that is based on Apache Arrow 7.0.0).

The following dependencies are missed from link command line:

• vcpkg_installed/x64-linux/debug/lib/libthriftd.a
• vcpkg_installed/x64-linux/debug/lib/liblz4d.a
• vcpkg_installed/x64-linux/debug/lib/libbrotli{enc,dec,common}-static.a

It seems that thrift::thrift, lz4::lz4 and unofficial::brotli::brotli{enc,dec,common}-static are missed in arrow_static dependencies:

$ grep INTERFACE_LINK_LIBRARIES vcpkg_installed/x64-linux/share/arrow/ArrowTargets.cmake 
  INTERFACE_LINK_LIBRARIES "OpenSSL::Crypto;OpenSSL::SSL;BZip2::BZip2;Snappy::snappy;ZLIB::ZLIB;zstd::libzstd_static;re2::re2;Threads::Threads;rt;\$<LINK_ONLY:>"

I'm not sure that this is a problem in Apache Arrow's CMake configuration or patches in vcpkg.

Issue Links

• links to: GitHub Pull Request #13863