thisisnic commented on a change in pull request #10710: URL: https://github.com/apache/arrow/pull/10710#discussion_r686575826
########## File path: cpp/cmake_modules/ThirdpartyToolchain.cmake ########## @@ -836,8 +836,18 @@ set(Boost_ADDITIONAL_VERSIONS # Thrift needs Boost if we're building the bundled version with version < 0.13, # so we first need to determine whether we're building it if(ARROW_WITH_THRIFT AND Thrift_SOURCE STREQUAL "AUTO") + find_package(PkgConfig QUIET) Review comment: @jonkeane It is, and it fixes an error on the build just before I added the change, though as @kou says, it should be moved -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org