Skip to content

Commit

Permalink
Merge pull request #58416 from cbodley/wip-cmake-zfs
Browse files Browse the repository at this point in the history
cmake/os: remove WITH_ZFS option

Reviewed-by: Igor Fedotov <[email protected]>
  • Loading branch information
cbodley authored Jul 12, 2024
2 parents 3f4aee2 + 52be40e commit dac3b7d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 47 deletions.
6 changes: 0 additions & 6 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -242,12 +242,6 @@ if(WITH_XFS)
set(HAVE_LIBXFS ${XFS_FOUND})
endif()

option(WITH_ZFS "enable LibZFS if found" OFF)
if(WITH_ZFS)
find_package(zfs)
set(HAVE_LIBZFS ${ZFS_FOUND})
endif()

option(WITH_BLUESTORE "Bluestore OSD backend" ON)
if(WITH_BLUESTORE)
if(LINUX)
Expand Down
28 changes: 0 additions & 28 deletions cmake/modules/Findzfs.cmake

This file was deleted.

13 changes: 0 additions & 13 deletions src/os/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,6 @@ if(HAVE_LIBXFS)
fs/XFS.cc)
endif()

if(HAVE_LIBZFS)
add_library(os_zfs_objs OBJECT
filestore/ZFSFileStoreBackend.cc
fs/ZFS.cc)
target_include_directories(os_zfs_objs SYSTEM PRIVATE
${ZFS_INCLUDE_DIRS})
list(APPEND libos_srcs $<TARGET_OBJECTS:os_zfs_objs>)
endif()

add_library(os STATIC ${libos_srcs})
target_link_libraries(os
legacy-option-headers
Expand All @@ -66,10 +57,6 @@ if(WITH_FUSE)
target_link_libraries(os FUSE::FUSE)
endif()

if(HAVE_LIBZFS)
target_link_libraries(os ${ZFS_LIBRARIES})
endif()

if(WITH_LTTNG)
add_dependencies(os objectstore-tp)
add_dependencies(os bluestore-tp)
Expand Down

0 comments on commit dac3b7d

Please sign in to comment.