Skip to content

Fix CMake generator expression (backport #30) - #31

Closed
mergify[bot] wants to merge 1 commit into
rollingfrom
mergify/bp/rolling/pr-30
Closed

Fix CMake generator expression (backport #30)#31
mergify[bot] wants to merge 1 commit into
rollingfrom
mergify/bp/rolling/pr-30

Conversation

@mergify

@mergify mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown

When building the kilted branch of micro-ROS/micro_ros_espidf_component, I get the following error:

--- stderr: rcl
CMake Error at CMakeLists.txt:88 (add_library):
  Target "rcl" links to target
  "$<0:rcl_interfaces::rcl_interfaces__rosidl_generator_c" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?


CMake Error at CMakeLists.txt:88 (add_library):
  Target "rcl" links to target
  "rcl_interfaces::rcl_interfaces__rosidl_typesupport_c>" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?


CMake Generate step failed.  Build files cannot be regenerated correctly.

This appears to be caused by unquoted generator expressions being split at semicolons. This issue only seems to occur with older versions of CMake. I'm using CMake 3.22.1.

Local testing confirms that the project builds successfully after the generator expressions are quoted.


This is an automatic backport of pull request #30 done by Mergify.

(cherry picked from commit dc07301)

# Conflicts:
#	rcl/CMakeLists.txt
@mergify mergify Bot added the conflicts label Aug 20, 2026
@mergify

mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Author

Cherry-pick of dc07301 has failed:

On branch mergify/bp/rolling/pr-30
Your branch is up to date with 'origin/rolling'.

You are currently cherry-picking commit dc07301.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   rcl/CMakeLists.txt

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify Bot mentioned this pull request Aug 20, 2026
@mergify

mergify Bot commented Aug 20, 2026

Copy link
Copy Markdown
Author

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

  • Queue this pull request

@David-LP99

Copy link
Copy Markdown

Change not needed in rolling

@David-LP99 David-LP99 closed this Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants