Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make fff installable #126

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

stephan-cr
Copy link

@stephan-cr stephan-cr commented Aug 17, 2023

This commit make fff installable, via cmake --install <build-dir> --prefix <some-install-prefix> for example. During installation, imported targets are created, such that users are able to use fff like this:

find_package(fff REQUIRED)

add_executable(some_test some_test.ccp)
target_link_libraries(some_test PRIVATE fff::fff)

This won't affect users who embed fff in their source trees.

Furthermore, if FFF_GENERATE is enable, fff.h is generated into the build folder, instead of overwriting the pre-generated fff.h.

Thank you for your contribution.

Before submitting this PR, please make sure:

  • Your code builds clean without any errors or warnings
  • You are not breaking consistency
  • You have added unit tests (doesn't apply)
  • All tests and other checks pass

This commit make fff installable, via `cmake --install <build-dir>
--prefix <some-install-prefix>` for example.  During installation,
imported targets are created, such that users are able to fff like
this:

```cmake
find_package(fff REQUIRED)

add_executable(some_test some_test.ccp)
target_link_libraries(some_test PRIVATE fff::fff)
´´´

This won't affect users who embed fff in their source trees,

Furthermore, if `FFF_GENERATE` is enable, fff.h is generated into the
build folder, instead of overwriting the pre-generated fff.h.
@stephan-cr
Copy link
Author

Not sure if you like the approach, but let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant