Skip to content

Add support for rclpy_message_converter and rclpy_message_converter_msgs in lyrical. - #43

Open
TomNysWF wants to merge 2 commits into
RoboStack:mainfrom
TomNysWF:add-rclpy-message-converter
Open

Add support for rclpy_message_converter and rclpy_message_converter_msgs in lyrical.#43
TomNysWF wants to merge 2 commits into
RoboStack:mainfrom
TomNysWF:add-rclpy-message-converter

Conversation

@TomNysWF

@TomNysWF TomNysWF commented Sep 8, 2026

Copy link
Copy Markdown

Add support for rclpy_message_converter and rclpy_message_converter_msgs in lyrical.

@TomNysWF

TomNysWF commented Sep 9, 2026

Copy link
Copy Markdown
Author

I had to add a different exclusion for the windows build to work, this same bug might also be present in other PR's

=>
The exclusion under packages_select_by_deps only kept it out of the top-level selection on win-64; it was still pulled in transitively as a dependency of autoware_internal_msgs/autoware_utils/autoware_utils_debug and failed there with MSVC error C1083 (long-path issue), aborting the Windows build before it ever reached the newly added rclpy_message_converter packages. Move it to packages_skip_by_deps/packages_remove_from_deps for win, matching the existing pendulum_control/rttest/tlsf pattern - it's only an unused package.xml of autoware_internal_msgs, not something its own .msg/CMake code actually needs.

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