Skip to content

Commit 245e4d9

Browse files
authored
Clean up a couple of CMake includes (#4710)
CMakeParseArguments is a no-op since CMake 3.5 and the minimum version {fmt} supports is 3.8: https://cmake.org/cmake/help/latest/module/CMakeParseArguments.html GNUInstallDirs is already unconditionally included earlier in the file.
1 parent 2b4eb7d commit 245e4d9

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

CMakeLists.txt

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -326,8 +326,6 @@ if (FMT_SAFE_DURATION_CAST)
326326
target_compile_definitions(fmt PUBLIC FMT_SAFE_DURATION_CAST)
327327
endif ()
328328

329-
include(CMakeParseArguments)
330-
331329
# Adds a library compiled with C++20 module support.
332330
#
333331
# Usage:
@@ -549,7 +547,6 @@ function (add_doc_target)
549547
--site-dir ${CMAKE_CURRENT_BINARY_DIR}/doc-html --no-directory-urls
550548
SOURCES ${sources})
551549

552-
include(GNUInstallDirs)
553550
install(
554551
DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/doc-html/
555552
DESTINATION ${CMAKE_INSTALL_DATAROOTDIR}/doc/fmt

0 commit comments

Comments
 (0)