Claus Klein
Claus Klein
> I think it might be enough to add the `SYSTEM` property to [cpm_add_subdirectory](https://github.com/cpm-cmake/CPM.cmake/blob/09b056ae20ea5590c4a08357fe3711e071ed7a8d/cmake/CPM.cmake#L961), however I'm currently not sure if there are any implications we should think about before implementing...
> @ClausKlein any thoughts / progress on moving your fork to the [CMakePackageRegistry](https://github.com/CMakePackageRegistry) org? No, at the moment I am working on boost v1.81.0 ... https://github.com/ClausKlein/boost-cmake/pull/6
Please note [Why you should NOT use Boost git repo with CPM.cmake](https://github.com/cpm-cmake/CPM.cmake/discussions/448) and this notes about [building-boost-with-cmake ](https://github.com/boostorg/cmake/tree/69f16e2819c4686450bdac807e3eb58346fb2ac2#building-boost-with-cmake)
I give it up
Note too the issue about [INTERFACE_SYSTEM_INCLUDE_DIRECTORIES the `${_IMPORT_PREFIX}/...` is missing!](https://gitlab.kitware.com/cmake/cmake/-/issues/23393)
### Sorry to say it, but there are still issues to fix: ```diff bash-3.2$ git diff ACE TAO diff --git a/ACE/tests/Reactor_Remove_Resume_Test_Dev_Poll.cpp b/ACE/tests/Reactor_Remove_Resume_Test_Dev_Poll.cpp index e591a9b0159..61a46779914 100644 --- a/ACE/tests/Reactor_Remove_Resume_Test_Dev_Poll.cpp +++ b/ACE/tests/Reactor_Remove_Resume_Test_Dev_Poll.cpp @@...
And deprecated warnings on **OSX** [build-macosx.log](https://github.com/DOCGroup/ACE_TAO/files/12380954/build-macosx.log)
> Feel free to open a pull request with the proposed changes @ClausKlein https://github.com/DOCGroup/ACE_TAO/pull/2097
The use case is that a header only project may check that every header is self contented and compiles. Too it is possible to `run-clang-tidy` for each header, because `CMake`...
> So if I understand correctly from the docs, the advantage is that we don't need to specify the include destination explicitly but can now use the `BASE_DIRS` flag in...