Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove architecture_independent from leo_msgs #1

Merged
merged 1 commit into from May 25, 2022
Merged

Remove architecture_independent from leo_msgs #1

merged 1 commit into from May 25, 2022

Conversation

cottsay
Copy link
Contributor

@cottsay cottsay commented May 24, 2022

In ROS 1, message packages did not create any binaries and were therefore architecture-independent packages.

In ROS 2, static typesupport results in library creation whenever messages are generated, which are not architecture-independent.

This should resolve the current build failure for building this package for RHEL: https://build.ros2.org/job/Hbin_rhel_el864__leo_msgs__rhel_8_x86_64__binary/

In ROS 1, message packages did not create any binaries and were
therefore architecture-independent packages.

In ROS 2, static typesupport results in library creation whenever
messages are generated, which are not architecture-independent.
@cottsay
Copy link
Contributor Author

cottsay commented May 24, 2022

Note that this PR is against the default branch of this repository, but the fix will be needed in humble. Please switch the branch of the PR if you prefer to merge the change there first.

@Bitterisland6 Bitterisland6 requested a review from bjsowa May 24, 2022 19:03
@bjsowa
Copy link
Member

bjsowa commented May 25, 2022

Thanks for the quick reaction. I was not aware about that (I'm still new to ROS2). I'll release the new version with this change tofoxy and humble.

btw. the buildfarm did not send any email about the failing job to the package maintainer. I don't know if that is the intended behavior.

@bjsowa bjsowa merged commit 48ba1f5 into LeoRover:foxy May 25, 2022
@cottsay cottsay deleted the msg-arch-pkg branch July 9, 2022 11:43
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.

None yet

2 participants