coffee/fmt
1
0
Fork 0
mirror of https://github.com/fmtlib/fmt.git synced 2026-07-30 16:26:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

Simplify build-tree export and rename INSTALL_TARGETS

Browse Source 
Rename INSTALL_TARGETS to FMT_TARGETS since the list is now used for both
installation and the build-tree export, and move INSTALL_FILE_SET into the
FMT_INSTALL block as only install(TARGETS) uses it.

Replace the export-test static library and export-lib.cc with an INTERFACE
library, which reproduces the same CMake export dependency check (#4806).
This commit is contained in:
Victor Zverovich 2026-07-22 11:24:25 -07:00
parent bcaa44d055
commit 4e5ff510f8
3 changed files with 14 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
CMakeLists.txt
View File

@ -442,13 +442,10 @@ set_target_properties(fmt-c PROPERTIES PUBLIC_HEADER include/fmt/fmt-c.h)
# Install targets.
set(targets_export_name fmt-targets)
set(INSTALL_TARGETS fmt fmt-header-only fmt-c)
set(FMT_TARGETS fmt fmt-header-only fmt-c)
if (FMT_MODULE)
list(APPEND INSTALL_TARGETS fmt-module)
if (FMT_USE_CMAKE_MODULES)
set(INSTALL_FILE_SET FILE_SET fmt DESTINATION ${FMT_INC_DIR}/fmt)
endif ()
list(APPEND FMT_TARGETS fmt-module)
endif ()
if (FMT_INSTALL)
@ -495,8 +492,11 @@ if (FMT_INSTALL)
INSTALL_DESTINATION ${FMT_CMAKE_DIR})
# Install the library and headers.
if (FMT_MODULE AND FMT_USE_CMAKE_MODULES)
set(INSTALL_FILE_SET FILE_SET fmt DESTINATION ${FMT_INC_DIR}/fmt)
endif ()
install(
TARGETS ${INSTALL_TARGETS}
TARGETS ${FMT_TARGETS}
COMPONENT fmt_core
EXPORT ${targets_export_name}
LIBRARY DESTINATION ${FMT_LIB_DIR}
@ -521,12 +521,12 @@ if (FMT_INSTALL)
COMPONENT fmt_core)
endif ()
# Generate an export file in the build tree so that fmt can be used by
# add_subdirectory() or FetchContent without being installed. Use a
# namespace because CMake provides better diagnostics for namespaced
# imported targets.
# Generate a build-tree export so that targets depending on fmt can also be
# exported without installing fmt, including when fmt is provided through
# add_subdirectory() or FetchContent. Use a namespace because CMake provides
# better diagnostics for namespaced imported targets.
export(
TARGETS ${INSTALL_TARGETS}
TARGETS ${FMT_TARGETS}
NAMESPACE fmt::
FILE ${PROJECT_BINARY_DIR}/${targets_export_name}.cmake)

8
test/add-subdirectory-test/CMakeLists.txt
View File

@ -16,11 +16,9 @@ if (TARGET fmt::fmt-header-only)
target_link_libraries(header-only-test fmt::fmt-header-only)
endif ()
# Test that a target publicly depending on fmt can itself be exported even
# though fmt is added via add_subdirectory and FMT_INSTALL is off (#4806).
add_library(export-test STATIC export-lib.cc)
target_link_libraries(export-test PUBLIC fmt::fmt)
# Test that a target depending on fmt can itself be exported (#4806).
add_library(export-test INTERFACE)
target_link_libraries(export-test INTERFACE fmt::fmt)
export(
TARGETS export-test
NAMESPACE test::
FILE ${CMAKE_CURRENT_BINARY_DIR}/export-test-targets.cmake)

3
test/add-subdirectory-test/export-lib.cc
View File

@ -1,3 +0,0 @@
#include "fmt/base.h"
void greet(const char* name) { fmt::print("Hello, {}!\n", name); }