search

dougbinks / enkiTS

A permissively licensed C and C++ Task Scheduler for creating parallel programs. Requires C++11 support.
zlib License
1.74k stars 145 forks source link

CMake use target_include_directories #76

Closed leonvictor closed 2 years ago

leonvictor commented 2 years ago

Hi, I forked the lib to be able to include enkiTS from my own project's CMakeLists. I see you where considering doing it yourself (#74) so here's a short PR :)

The behavior is essentially the same, this is just the newish recommended way to include directories and propagate them along with the CMake target. I've moved down the line so it is effective for shared and static builds.

dougbinks commented 2 years ago

This looks pretty simple - I'll double check and merge shortly.

dougbinks commented 2 years ago

Merged, many thanks for the PR!

leonvictor commented 2 years ago

Thank you for the speedy process and amazing lib ! :)