This is an automated email from the ASF dual-hosted git repository.
gangwu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-cpp.git
The following commit(s) were added to refs/heads/main by this push:
new e7f1d0f5 fix(cmake): require Parquet when using system Arrow (#541)
e7f1d0f5 is described below
commit e7f1d0f539eb9586319cb9115e47e395e8377aea
Author: liuxiaoyu <[email protected]>
AuthorDate: Wed Jan 28 21:28:44 2026 +0800
fix(cmake): require Parquet when using system Arrow (#541)
When using system Arrow, ensure Parquet is also discovered so
Parquet::parquet_* targets are available for iceberg_bundle linking.
This restores find_package(Parquet) and adds Parquet to
ICEBERG_SYSTEM_DEPENDENCIES to avoid missing target errors.
---
cmake_modules/IcebergThirdpartyToolchain.cmake | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/cmake_modules/IcebergThirdpartyToolchain.cmake
b/cmake_modules/IcebergThirdpartyToolchain.cmake
index dd1f8a87..27745936 100644
--- a/cmake_modules/IcebergThirdpartyToolchain.cmake
+++ b/cmake_modules/IcebergThirdpartyToolchain.cmake
@@ -151,7 +151,9 @@ function(resolve_arrow_dependency)
endif()
else()
set(ARROW_VENDORED FALSE)
- list(APPEND ICEBERG_SYSTEM_DEPENDENCIES Arrow)
+ find_package(Arrow CONFIG REQUIRED)
+ find_package(Parquet CONFIG REQUIRED)
+ list(APPEND ICEBERG_SYSTEM_DEPENDENCIES Arrow Parquet)
endif()
set(ICEBERG_SYSTEM_DEPENDENCIES