Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Cmake now fails when an option is enabled but a needed dependency is missing.
Details
To not fail too much, only the minimal set of options to use cucumber-cpp is enabled by default.
Test frameworks are off by default because every one has an external dependency. A warning is shown when no test framework is selected as one is needed for cucumber-cpp to be usable.
Motivation and Context
When a developer enables an option she want to use this feature. At the moment, the option can be enabled but when the dependency is not found that feature is not included.
With this change, cmake fails if a dependency for an enabled option is not found.
How Has This Been Tested?
Manually by configuring the project with cmake. Changes in the CI script also reflect the changes.
Types of changes
This change possibly breaks some existing CI because no test framework is selected by default. The warning message shows what the problem is and how it can be solved.
Checklist: