search

rollbear / trompeloeil

Header only C++14 mocking framework
Boost Software License 1.0
811 stars 86 forks source link

CMake target namespace #242

Closed offa closed 2 years ago

offa commented 3 years ago

If installed through CMake trompeloeil lacks a target trompeloeil::trompeloeil.

How about moving it to a namespace? That way it'll provide the mentioned target — unfortunately no plain trompeloeil anymore I guess though.

rollbear commented 3 years ago

So weird. I was pretty sure this was added earlier this year. Commit 7bff1a432929bfada846a8d65554b77fe8ba7553 implies it, for sure.

Today is not the right day for me to dive into this, but I'll get back to you shortly. If you want to do git archaeology and find out where I absent minded that commit away, it'd be splendid. Thanks for reporting.

offa commented 3 years ago

Seems alias targets don't get installed. I can provide a PR as adding the namespace isn't a big deal.

rollbear commented 3 years ago

Please do. Thanks. Much appreciated!

rollbear commented 3 years ago

Merged. Let's leave this open until the next release is tagged. Thanks for both the report and the fix.

rollbear commented 2 years ago

Fix included in release v42