Skip to content
This repository was archived by the owner on Feb 26, 2025. It is now read-only.

Allow disabling coverage for 'Debug'.#299

Merged
joni-herttuainen merged 1 commit intomasterfrom
1uc/optional-coverage
Nov 3, 2023
Merged

Allow disabling coverage for 'Debug'.#299
joni-herttuainen merged 1 commit intomasterfrom
1uc/optional-coverage

Conversation

@1uc
Copy link
Copy Markdown
Collaborator

@1uc 1uc commented Oct 25, 2023

If the user specifies -DSONATA_ENABLE_COVERAGE then that choice is respected. If no choice is made coverage is On for Debug build; and Off otherwise.

Turning off coverage is useful, if one doesn't have lcov but still wants a debug build.

@1uc 1uc marked this pull request as ready for review October 25, 2023 13:53
@1uc 1uc changed the title Allow disabling coverage an 'Debug'. Allow disabling coverage for 'Debug'. Nov 2, 2023
If the user specifies `-DSONATA_ENABLE_COVERAGE` then that choice is
respected. If no choice is made coverage is `On` for `Debug` build; and
`Off` otherwise.

Turning off coverage is useful, if one doesn't have `lcov` but still
wants a debug build.
@joni-herttuainen joni-herttuainen merged commit 6da4886 into master Nov 3, 2023
@joni-herttuainen joni-herttuainen deleted the 1uc/optional-coverage branch November 3, 2023 09:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants