tadeja commented on PR #51280: URL: https://github.com/apache/arrow/pull/51280#issuecomment-5733789506
> Copilot Needs a closer look > Add static pkg-config coverage or assertions for the Azure/GCS private dependencies. I've locally checked pkg-config metadata with a static build (`-DARROW_FILESYSTEM=ON -DARROW_S3=ON -DARROW_GCS=ON -DARROW_AZURE=ON -DARROW_BUILD_STATIC=ON -DAzure_SOURCE=BUNDLED -Dgoogle_cloud_cpp_storage_SOURCE=BUNDLED`), ``` grep "^Requires.private" src/arrow/release/arrow.pc ``` luckily showed duplication - commit 9ab926d resolves it. Now the correct result is: `Requires.private: zlib libcurl libxml-2.0` Further CI check for downstream static bundled Azure/GCS is out of scope here I think, potentially a follow-up if ever needed https://github.com/apache/arrow/blob/c07de67a7c9a43c7c958ffeb08407b66c38df5b5/ci/scripts/cpp_test.sh#L126 -- 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]
