Skip to content

Commit

Permalink
Only install FILE_SET when needed (#4013)
Browse files Browse the repository at this point in the history
  • Loading branch information
yujincheng08 committed Jun 13, 2024
1 parent 3e3062c commit c4ea903
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -403,13 +403,19 @@ if (FMT_INSTALL)

set(INSTALL_TARGETS fmt fmt-header-only)

if (CMAKE_VERSION VERSION_GREATER_EQUAL 3.28 AND CMAKE_GENERATOR STREQUAL "Ninja")
set(INSTALL_FILE_SET FILE_SET fmt DESTINATION "${FMT_INC_DIR}/fmt")
else()
set(INSTALL_FILE_SET)
endif()

# Install the library and headers.
install(TARGETS ${INSTALL_TARGETS} EXPORT ${targets_export_name}
LIBRARY DESTINATION ${FMT_LIB_DIR}
ARCHIVE DESTINATION ${FMT_LIB_DIR}
PUBLIC_HEADER DESTINATION "${FMT_INC_DIR}/fmt"
FILE_SET fmt DESTINATION "${FMT_INC_DIR}/fmt"
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
${INSTALL_FILE_SET})

# Use a namespace because CMake provides better diagnostics for namespaced
# imported targets.
Expand Down

0 comments on commit c4ea903

Please sign in to comment.