Skip to content

Commit 91ac34f

Browse files
committed
Clean up a couple of CMake includes
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 91ac34f

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

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)