search

eranpeer / FakeIt

C++ mocking made easy. A simple yet very expressive, headers only library for c++ mocking.
MIT License
1.22k stars 169 forks source link

Add cmake support. #192

Closed libenike closed 1 year ago

libenike commented 4 years ago

Replace old CMakeLists.txt with the new one. Now FakeIt can be linked to the project in the following way:

find_package(FakeIt)
...
target_link_libraries(a_project
    PRIVATE
        FakeIt::FakeIt
)

I found a related issue with this request. Also, I create a repository with a simple project which uses FakeIt and a detailed description of building the project.

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 99.923% when pulling 08d41c69a83f1e9506a2335a572a9b78c625e7f2 on libenike:feature/add_cmake_support into 317419c2e2f5a98e023a4d62628eb149fe3d3c3a on eranpeer:master.

aboseley commented 3 years ago

Whats the status of this pull request?

libenike commented 3 years ago

Whats the status of this pull request?

The status is open. I am waiting for any response from contributors. You are welcome to leave any suggestions about my fork.

aboseley commented 3 years ago

@libenike , your branch works for me, I'd like to see this merged to master.

FranckRJ commented 2 years ago

I've thought about it, wouldn't it be better to have one target per configuration? With explicit names I guess, so no "FakeIt::FakeIt" but instead "FakeIt::standalone" or something like that.

FranckRJ commented 1 year ago

I've updated some things in this PR: https://github.com/eranpeer/FakeIt/pull/311

I removed the target FakeIt::FakeIt, it was replaced by FakeIt::FakeIt-standalone and there's one target per configurations. I guess it's better to install everything even if it makes names a bit more complicated, but maybe we can add a "default configuration" option which will create a target FakeIt::FakeIt to keep the name clean. I don't know yet, it can always be added latter anyway.