-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Labels
help wantedExtra attention is neededExtra attention is needed📦 distributionPackage managers, build systems, repositories... everything to simplify consumption of the libraryPackage managers, build systems, repositories... everything to simplify consumption of the library
Description
Hi @an-dr - quick question regarding the way to use the lib with CMake.
In the README, it is said that "Option 2 - CMake Package (recommended CMake > 3.15.0)".
In my project, the preferred approach is to have the dependencies integrated as submodules. Then, thanks to the root CMakeLists of the different projects, I can easily integrate them,
In the current CMakeLists, we can see that the library is declared as an INTERFACE, which is not an expected pattern because it shifts the responsibility of compiling the core logic from the library itself to its consumer.
What do you think about changing it to STATIC instead?
Cheers.
an-dr
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is needed📦 distributionPackage managers, build systems, repositories... everything to simplify consumption of the libraryPackage managers, build systems, repositories... everything to simplify consumption of the library