-
-
Notifications
You must be signed in to change notification settings - Fork 620
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
Suggestion: document a simpler CMake integration via FetchContent #823
Comments
Ah, my issue search skills are obviously pretty bad, thank you! Thank you! |
Currently in build-systems.md the suggested CMake integrations are either via manual file download or
ExternalProject
.The first is, well, manual and the second is heavyweight, cumbersome and downloads at build time (yikes!).
There is a much simpler way to integrate doctest using FetchContent.
Something like this:
This downloads just the header and does it during configure, not build, time.
If you think this is reasonable to document in addition or instead of
ExternalProject
method happy to submit a PR.The text was updated successfully, but these errors were encountered: