CMake: cross-platform Webots controller build (macOS prebuilt vs Linux source) #1046
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR improves the CMake build of
webots_ros2_driverto support both Linux and macOS environments:/Applications/Webots.appon macOS and link against its libraries.file(GLOB …)for Webots libraries instead of hardcoded list.humble/iron → 3.10, others → 3.12).yaml-cpptoyaml_cpp_vendorfor consistency with ROS 2.add_dependenciesonly when not using prebuilt Webots.webots-controllerscript installation now uses${WEBOTS_CONTROLLER_EXEC}.Motivation
These changes allow
webots_ros2_driverto build on:Notes
@traversaro — I would greatly appreciate your guidance and feedback here. :)
Testing