cmake: no libradosstriper headers if WITH_LIBRADOSSTRIPER=OFF

This commit conditionalizes inclusion of the libradosstriper header files and
moves the "option(WITH_LIBRADOSSTRIPER..)" directive to before
"add_subdirectory(include)", as we check for this variable in src/include.

Fixes: http://tracker.ceph.com/issues/35922
Signed-off-by: Kefu Chai <kchai@redhat.com>
Signed-off-by: Nathan Cutler <ncutler@suse.com>

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 7380a92906afa6e49547b6b855d471df46d75ca1..3d638fc3846647cdc57f9449125d0502399c2842 100644 (file)
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -438,10 +438,11 @@ add_subdirectory(perfglue)
 
 add_library(rados_snap_set_diff_obj OBJECT librados/snap_set_diff.cc)
 
+option(WITH_LIBRADOSSTRIPER "build with libradosstriper support" ON)
+
 add_subdirectory(include)
 add_subdirectory(librados)
 
-option(WITH_LIBRADOSSTRIPER "build with libradosstriper support" ON)
 if(WITH_LIBRADOSSTRIPER)
   add_subdirectory(libradosstriper)
 endif()

diff --git a/src/include/CMakeLists.txt b/src/include/CMakeLists.txt
index 581b3ebe03ef4ddb780678f130eae8ea17b60222..b2eafc917a250bf8664f933ed18da40e1c00f357 100644 (file)
--- a/src/include/CMakeLists.txt
+++ b/src/include/CMakeLists.txt
@@ -9,10 +9,12 @@ install(FILES rados/librados.h
   crc32c.h
   rados/objclass.h
   DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/rados)
-install(FILES
-  radosstriper/libradosstriper.h
-  radosstriper/libradosstriper.hpp
-  DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/radosstriper)
+if(WITH_LIBRADOSSTRIPER)
+  install(FILES
+    radosstriper/libradosstriper.h
+    radosstriper/libradosstriper.hpp
+    DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/radosstriper)
+endif()
 
 if(WITH_RBD)
   install(FILES