Installing to libexec with ament_cmake_auto about ament_cmake HOT 6 CLOSED

It would definitely be good if this CMake function would have an option to choose the install destination of the executables (either bin or lib/<pkgname>). For ROS packages the second option looks like the better default, too. If we want to keep the default in this package as bin we could also introduce a ROS specific package (like ament_cmake_ros) but I don't feel strongly about a separate package for this.

from ament_cmake.

ok, so how about we change the default for executables to lib/${PROJECT_NAME} and provide and option called INSTALL_TO_PATH or something that installs executables how they are installed currently?

from ament_cmake.

Ah, I noticed in the FHS standard that "Applications may use a single subdirectory under /usr/lib. If an application uses a subdirectory, all architecture-dependent data exclusively used by the application must be placed within that subdirectory."

Does this then mean that if the executables are being installed to lib/${PROJECT_NAME}, the libraries should be installed there too? This is not consistent with how we've been changing things for other ROS 2 packages so I may have misinterpreted 'exclusively used by the application'.

from ament_cmake.

Libraries should still go into lib. This is how we do it in ROS 1 and I don't think we should change that.

from ament_cmake.

Ah, I noticed in the FHS standard that "Applications may use a single subdirectory under /usr/lib. If an application uses a subdirectory, all architecture-dependent data exclusively used by the application must be placed within that subdirectory."

I think the key word there is "exclusively", so if you have a shared library that only your application uses, it would be put there. Since we mostly make things that could be reused by other packages, those shared libraries still belong in the top-level lib. At least that's how I interpret it.

from ament_cmake.

closed bu #97, thanks @wjwwood and @dirk-thomas for the input

from ament_cmake.