Skip to content

Move export() directive outside of FMT_INSTALL scope(fixed CI)#4831

Closed
RealSteel99-codes wants to merge 1 commit into
fmtlib:mainfrom
RealSteel99-codes:main
Closed

Move export() directive outside of FMT_INSTALL scope(fixed CI)#4831
RealSteel99-codes wants to merge 1 commit into
fmtlib:mainfrom
RealSteel99-codes:main

Conversation

@RealSteel99-codes

Copy link
Copy Markdown

Closes #4806

Comment thread CMakeLists.txt
Comment on lines +508 to +515
# imported targets.
export(
TARGETS ${INSTALL_TARGETS}
AMESPACE fmt::
FILE ${PROJECT_BINARY_DIR}/${targets_export_name}.cmake)

# Install version, config and target files.
install(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks very broken.

@vitaut

vitaut commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Please fix the original PR instead of creating a new one.

@vitaut vitaut closed this Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants