support building separate debug and release targets about ament_cmake HOT 4 CLOSED

I think this is in the responsibility of each packages CMake code to use build type specific suffixes (and make sure to export them accordingly). I am not sure how ament_cmake could do that generically.

Also if you build a workspace with e.g. CMAKE_BUILD_TYPE=Debug you should be able to leverage the full potential of the Visual Studio debugger without the need for any changes.

from ament_cmake.

Maybe, this isn't the best place to put this request. I mostly want to see all the ROS 2 packages in https://github.com/ros2/ros2/blob/master/ros2.repos have standard library suffixes based on CMAKE_BUILD_TYPE. That way the libraries can be built in Debug mode and Release mode without clobbering the other artifacts.

ament_target_dependencies would need know which artifact to chose. Also, the ROS 2 Windows binary should include both the debug and release libraries.

from ament_cmake.

If a package builds both library types at the same type it can export something like this: release libfoo debug libfood. But that is rather uncommon for a CMake project. If a package only builds one build type at a type it can also only export that library.

So I am not sure how you would like to achieve this. Maybe it would be best to propose a few PRs to some lower level packages. And if the approach works well the same approach can be considered for all other packages.

from ament_cmake.

@geoffviola Closing this due to inactivity and lack of a clear path forward. Feel free to keep commenting on the issue and we can reopen it

from ament_cmake.