RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2024-12-27 22:13:28 +00:00
Code Issues Packages Projects Releases Wiki Activity
1a9f304d2b
ceph/cmake/modules/FindStdFilesystem.cmake

52 lines
CMake
Raw Normal View History

cmake: extract std::filesystem linkage checking into module see https://libcxx.llvm.org/docs/UsingLibcxx.html#using-libc-experimental-and-experimental, > Note that as of libc++ 7.0 using the <experimental/filesystem> > requires linking libc++fs instead of libc++experimental. do not build ceph_test_admin_socket_output if we are not able to find the library for std::experimental::filesystem . it is a workaround of https://reviews.freebsd.org/D10840 where FreeBSD 11.2 does not ship libc++experimental.a . Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-07-28 02:36:20 +00:00
set(_std_filesystem_test_src
${CMAKE_CURRENT_LIST_DIR}/FindStdFilesystem_test.cc)
macro(try_std_filesystem_library _library _result)
cmake: fix StdFilesystem detection before this change, we fails to detect std::filesystem with clang++, because the cmake project created by try_compile() only expands following options passed from its caller: - COMPILE_DEFINITIONS - INCLUDE_DIRECTORIES - LINK_DIRECTORIES - LINK_LIBRARIES which do not include CMAKE_CXX_FLAGS, so either we need to (ab)use COMPILE_DEFINITIONS for passing -std=c++17, or we can change the CMAKE_CXX_FLAGS in the parent env, as it turns out the created cmake project does inherit this flag from current project. in this change, we use the COMPILE_DEFINITIONS approach: simpler this way. and we can drop it once cmake 3.8 is required. Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-08-02 05:45:51 +00:00
if(CMAKE_VERSION VERSION_LESS "3.8")
# abuse the definition flags, because they are quite
# the same as CMAKE_C_FLAGS: they are passed to the
# compiler.
set(_std_filesystem_try_compile_arg
COMPILE_DEFINITIONS "-std=c++17")
else()
set(_std_filesystem_try_compile_arg
CXX_STANDARD 17)
endif()
cmake: extract std::filesystem linkage checking into module see https://libcxx.llvm.org/docs/UsingLibcxx.html#using-libc-experimental-and-experimental, > Note that as of libc++ 7.0 using the <experimental/filesystem> > requires linking libc++fs instead of libc++experimental. do not build ceph_test_admin_socket_output if we are not able to find the library for std::experimental::filesystem . it is a workaround of https://reviews.freebsd.org/D10840 where FreeBSD 11.2 does not ship libc++experimental.a . Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-07-28 02:36:20 +00:00
try_compile(_std_filesystem_compiles
${CMAKE_CURRENT_BINARY_DIR}
SOURCES ${_std_filesystem_test_src}
cmake: fix StdFilesystem detection before this change, we fails to detect std::filesystem with clang++, because the cmake project created by try_compile() only expands following options passed from its caller: - COMPILE_DEFINITIONS - INCLUDE_DIRECTORIES - LINK_DIRECTORIES - LINK_LIBRARIES which do not include CMAKE_CXX_FLAGS, so either we need to (ab)use COMPILE_DEFINITIONS for passing -std=c++17, or we can change the CMAKE_CXX_FLAGS in the parent env, as it turns out the created cmake project does inherit this flag from current project. in this change, we use the COMPILE_DEFINITIONS approach: simpler this way. and we can drop it once cmake 3.8 is required. Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-08-02 05:45:51 +00:00
LINK_LIBRARIES ${_library}
${_std_filesystem_try_compile_arg})
unset(_std_filesystem_try_compile_arg)
cmake: extract std::filesystem linkage checking into module see https://libcxx.llvm.org/docs/UsingLibcxx.html#using-libc-experimental-and-experimental, > Note that as of libc++ 7.0 using the <experimental/filesystem> > requires linking libc++fs instead of libc++experimental. do not build ceph_test_admin_socket_output if we are not able to find the library for std::experimental::filesystem . it is a workaround of https://reviews.freebsd.org/D10840 where FreeBSD 11.2 does not ship libc++experimental.a . Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-07-28 02:36:20 +00:00
if(_std_filesystem_compiles)
set(${_result} ${_library})
endif()
cmake: fix StdFilesystem detection before this change, we fails to detect std::filesystem with clang++, because the cmake project created by try_compile() only expands following options passed from its caller: - COMPILE_DEFINITIONS - INCLUDE_DIRECTORIES - LINK_DIRECTORIES - LINK_LIBRARIES which do not include CMAKE_CXX_FLAGS, so either we need to (ab)use COMPILE_DEFINITIONS for passing -std=c++17, or we can change the CMAKE_CXX_FLAGS in the parent env, as it turns out the created cmake project does inherit this flag from current project. in this change, we use the COMPILE_DEFINITIONS approach: simpler this way. and we can drop it once cmake 3.8 is required. Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-08-02 05:45:51 +00:00
unset(_std_filesystem_compiles)
cmake: extract std::filesystem linkage checking into module see https://libcxx.llvm.org/docs/UsingLibcxx.html#using-libc-experimental-and-experimental, > Note that as of libc++ 7.0 using the <experimental/filesystem> > requires linking libc++fs instead of libc++experimental. do not build ceph_test_admin_socket_output if we are not able to find the library for std::experimental::filesystem . it is a workaround of https://reviews.freebsd.org/D10840 where FreeBSD 11.2 does not ship libc++experimental.a . Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-07-28 02:36:20 +00:00
endmacro()
if(NOT StdFilesystem_LIBRARY)
try_std_filesystem_library("stdc++fs" StdFilesystem_LIBRARY)
endif()
if(NOT StdFilesystem_LIBRARY)
try_std_filesystem_library("c++experimental" StdFilesystem_LIBRARY)
endif()
if(NOT StdFilesystem_LIBRARY)
try_std_filesystem_library("c++fs" StdFilesystem_LIBRARY)
endif()
cmake: fix StdFilesystem detection before this change, we fails to detect std::filesystem with clang++, because the cmake project created by try_compile() only expands following options passed from its caller: - COMPILE_DEFINITIONS - INCLUDE_DIRECTORIES - LINK_DIRECTORIES - LINK_LIBRARIES which do not include CMAKE_CXX_FLAGS, so either we need to (ab)use COMPILE_DEFINITIONS for passing -std=c++17, or we can change the CMAKE_CXX_FLAGS in the parent env, as it turns out the created cmake project does inherit this flag from current project. in this change, we use the COMPILE_DEFINITIONS approach: simpler this way. and we can drop it once cmake 3.8 is required. Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-08-02 05:45:51 +00:00
unset(_std_filesystem_test_src)
cmake: extract std::filesystem linkage checking into module see https://libcxx.llvm.org/docs/UsingLibcxx.html#using-libc-experimental-and-experimental, > Note that as of libc++ 7.0 using the <experimental/filesystem> > requires linking libc++fs instead of libc++experimental. do not build ceph_test_admin_socket_output if we are not able to find the library for std::experimental::filesystem . it is a workaround of https://reviews.freebsd.org/D10840 where FreeBSD 11.2 does not ship libc++experimental.a . Signed-off-by: Kefu Chai <kchai@redhat.com>
2018-07-28 02:36:20 +00:00
include(FindPackageHandleStandardArgs)
find_package_handle_standard_args(StdFilesystem
FOUND_VAR StdFilesystem_FOUND
REQUIRED_VARS StdFilesystem_LIBRARY)
mark_as_advanced(StdFilesystem_LIBRARY)
if(StdFilesystem_FOUND AND NOT (TARGET StdFilesystem::filesystem))
add_library(StdFilesystem::filesystem INTERFACE IMPORTED)
set_target_properties(StdFilesystem::filesystem PROPERTIES
INTERFACE_LINK_LIBRARIES ${StdFilesystem_LIBRARY})
endif()
Reference in New Issue Copy Permalink