fmtlib/fmt

`export(` should not be scoped by `FMT_INSTALL`

Open

#4806 aperta il 14 giu 2026

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (2881 fork)batch import
help wanted

Metriche repository

Star
 (23.489 star)
Metriche merge PR
 (Merge medio 2g 9h) (12 PR mergiate in 30 g)

Descrizione

Currently the export(...) cmake directive is scoped inside if (FMT_INSTALL): https://github.com/fmtlib/fmt/blob/11ddbcb7898d2d3445d431a54814367b21dee6ad/CMakeLists.txt#L496-L499

However, there are cases in which downstream projects might not want to install fmt but might still want to provide a export(...) themselves. However, this is impossible to do when depending on another target that doesn't export itself. CMake will error with something like

CMake Error: export("MyLibTargets" ...) includes target "mylib" which requires target "fmt" that is not in any export set.

Since having the exported target files in the build tree don't harm anyone, I suggest just unconditionally exporting regardless of whether FMT_INSTALL is true or not.

See also e.g. https://github.com/Kistler-Group/sdbus-cpp/pull/523

Guida contributor

Direzione di ricerca
Sposta la direttiva `export(...)` al di fuori del blocco `if (FMT INSTALL)` per consentire ai progetti downstream di esportare i target anche quando FMT INSTALL è disattivato.
Tech stack
cpp
Dominio
build system
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CMake
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70