meyerj
commented
5 years ago As there were no objections since 2017, we are going to merge this into toolchain-2.10 now, which will soon be merged into master as a whole.
Closed meyerj closed 5 years ago
meyerj
commented
5 years ago As there were no objections since 2017, we are going to merge this into toolchain-2.10 now, which will soon be merged into master as a whole.
...for consistency with log4cpp (since toolchain-2.9), ocl and the default include directory for Orocos packages (see orocos_install_headers() and orocos_generate_package() macros in UseOrocos-RTT.cmake).
The renaming has primarily practical reasons during RTT development, if one version of RTT is installed (e.g. from Debian package) while working on another, incompatible version in a custom workspace. If RTT installs its headers directly to
{prefix}/include/rtt(e.g./usr/include/rtt) it cannot be avoided that dependents find the installed headers instead, which is likely to cause linker errors or segmentation faults.This commit solves the problem at least for the case that RTT and all its dependents are present in the overlayed workspace and
{prefix}/include/orocosof the installed version is not in the header search path.Any objectives?
TODO:
include/rtt/typelib, which should also be changed toinclude/orocos/rtt/typelibthen (see rtt_typelib/CMakeLists.txt:46). All other Orocos packages I am currently aware of are already installing their headers toinclude/orocosor use theorocos_install_headers()macro without overriding the default destination directory.