]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cmake/rgw: add missing dependency on Arrow::Arrow
authorCasey Bodley <cbodley@redhat.com>
Thu, 28 Apr 2022 15:43:15 +0000 (11:43 -0400)
committerCasey Bodley <cbodley@redhat.com>
Thu, 28 Apr 2022 15:43:17 +0000 (11:43 -0400)
when WITH_SYSTEM_ARROW is disabled, BuildArrow.cmake creates an
Arrow::Parquet target that depends on Arrow::Arrow:

  target_link_libraries(Arrow::Parquet INTERFACE Arrow::Arrow)

but when WITH_SYSTEM_ARROW is enabled, the targets we get from
find_package() do not carry this dependency. so rgw's cmake needs to
depend on both targets

Fixes: https://tracker.ceph.com/issues/55420
Signed-off-by: Casey Bodley <cbodley@redhat.com>
src/rgw/CMakeLists.txt

index 5232108cfe50ef9cbd7ea377843edc5d94c04587..3f569deb20bbe4912edd407376720e35b687c44b 100644 (file)
@@ -4,7 +4,7 @@ if(NOT GPERF)
 endif()
 
 if(WITH_RADOSGW_SELECT_PARQUET)
-  set(ARROW_LIBRARIES Arrow::Parquet)
+  set(ARROW_LIBRARIES Arrow::Arrow Arrow::Parquet)
   add_definitions(-D_ARROW_EXIST)
   message("-- arrow is installed, radosgw/s3select-op is able to process parquet objects")
 endif(WITH_RADOSGW_SELECT_PARQUET)