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

Add hook to create ccov target each time that add_ex.. and add_lib...… #35

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rlalik
Copy link
Contributor

@rlalik rlalik commented May 5, 2022

… is called.

The main motivation for this change was laziness. If I need coverage for each and every target in my project I do not want to double every add_* function with extra to add coverage, I want cmake to do it for me.

rlalik added 3 commits May 5, 2022 21:45
IMPORTED and ALIAS targets cannot be add to coverage.
Also, in old project where PUBLIC, PRIVATE or INTERFACE visibility
was not specified on target, there is option PLAIN which do not
set any for compile and link (this requires PUBLIC) option.
@rlalik
Copy link
Contributor Author

rlalik commented May 6, 2022

Ok, I need to work on this a bit more. For sure my two other pull-request are also included in this PR (I later split it into separate) here so I would ask to accept these two first:

before proceeding with this. Then I make this PR smaller.

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