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

Reply via email to