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.

Sign up for GitHub

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

Jump to bottom

cmake fails if coverxygen is not found #26

Closed
kidder opened this issue May 30, 2017 · 1 comment
Closed

cmake fails if coverxygen is not found #26

kidder opened this issue May 30, 2017 · 1 comment
Assignees
@nilsdeppe
Labels
bug
Milestone
Version 0.0.1

Comments

@kidder
Copy link
Member

kidder commented May 30, 2017

Coverxygen is listed as an optional package and should not cause cmake to fail if it is not found
@kidder kidder added the bug label May 30, 2017
@kidder kidder added this to the Version 0.0.1 milestone May 30, 2017
@nilsdeppe nilsdeppe self-assigned this May 30, 2017
nilsdeppe added a commit to nilsdeppe/spectre that referenced this issue May 30, 2017
@nilsdeppe
Fix issue sxs-collaboration#26
bba5416 
Coverxygen was set as required but should be an optional dependency. Now check
if it was found and add the doc-coverage target only then.
nilsdeppe added a commit to nilsdeppe/spectre that referenced this issue May 30, 2017
@nilsdeppe
Fix issue sxs-collaboration#26
ae6ed2f 
Coverxygen was set as required but should be an optional dependency. Now check
if it was found and add the doc-coverage target only then.
kidder added a commit that referenced this issue May 30, 2017
@kidder
Merge pull request #27 from nilsdeppe/fix/issue_26
133692a 
Fix issue #26
@kidder kidder closed this as completed May 30, 2017
@kidder
Copy link
Member Author

kidder commented May 30, 2017

fixed in commit 133692a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nilsdeppe nilsdeppe

Labels
bug
Projects
None yet
Milestone
Version 0.0.1
Development

No branches or pull requests
2 participants
@kidder @nilsdeppe