Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

Adds CMake library and install target #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adds CMake library and install target #76

wants to merge 2 commits into from

Conversation

timothyqiu
Copy link

This PR

  • adds an interface library target Clara, aliased Clara::Clara

  • corresponding install entries are also added

    clara-install-prefix/
    |-- include
    |   `-- clara.hpp
    `-- lib
        `-- cmake
            `-- Clara
                |-- ClaraConfig.cmake
                `-- ClaraConfigVersion.cmake
    

    so find_package(Clara REQUIRED) can properly import Clara::Clara from the installed package.

  • include directories needed for ClaraTest targets are now set using target_include_directories instead of changing global settings

This implements the main part of #44

@timothyqiu
Copy link
Author

😞 One CI job failed because it can't pull code from GitHub, I don't know how to trigger that job again...

CMakeLists.txt Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 2, 2019

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   94.85%   94.85%           
=======================================
  Files           2        2           
  Lines         680      680           
=======================================
  Hits          645      645           
  Misses         35       35
Impacted Files Coverage Δ
include/clara.hpp 93.3% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6dfbbd...ba9db5b. Read the comment docs.

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.

2 participants