Skip to content

Fix 'microxrcedds_agent' name and dependency - #279

Open
David-LP99 wants to merge 2 commits into
rollingfrom
update_build
Open

Fix 'microxrcedds_agent' name and dependency#279
David-LP99 wants to merge 2 commits into
rollingfrom
update_build

Conversation

@David-LP99

Copy link
Copy Markdown
Contributor

Description

Before #237, the micro-ROS-Agent SuperBuild.cmake used to refer to the Micro-XRCE-DDS-Agent as xrceagent, which is inconsistent with the name exported by that package (microxrcedds_agent). In #237, the name and dependencies were updated, but not correctly (in line 24 the old name was kept), which caused some issues and was reverted on #239. The current approach works well for Vulcanexus, but can cause problems depending on the workspace configuration or build order.

This PR approaches this issue and tries to generate a consistent behaviour for building the package regardless of where the Micro-XRCE-DDS-Agent is coming from.

XRCE-DDS-Agent comes from What ensures correct build order
this repo's SuperBuild.cmake clones/builds it the DEPENDS ${_deps} (non-empty) at the end of SuperBuild.cmake
it is preinstalled in the system (Vulcanexus case) nothing to order, it is already there
another package in your workspace colcon, thanks to package.xml

Main changes

  • The references to xrceagent are renamed as microxrcedds_agent
  • The micro-ROS-Agent's package.xml now depends on microxrcedds_agent as well
  • When microxrcedds_agent is not found, it is added as a dependency to the micro-ROS-Agent external project in SuperBuild.cmake
  • microxrcedds_agent 3.0.2 or higher is needed, since that version starts exporting an EXACT dependency on spdlog 1.9.2, which ensures that the micro-ROS-Agent build doesn't choose a different version from somewhere else

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
@mergify

mergify Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

Signed-off-by: David Laseca Perez <davidlaseca@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant