-
Notifications
You must be signed in to change notification settings - Fork 161
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
[rcl lifecycle] removed rmw_implementation from package.xml #575
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, with CI.
I'd like @dirk-thomas to look at it as well though, in case I'm forgetting a reason why it is needed.
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
f1f2ad5
to
1b37026
Compare
rcl/rcl_lifecycle/src/com_interface.c Lines 34 to 35 in c601464
rosidl_generator_c so I don't think the dependency on it can be removed.
@ahcorde Please update the title (which will become the squashed commit message) to describe the actual change. As is it is not saying anything useful. |
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
unrelated failures. Merging |
rcl_lifecycle package
<build_depend>
and<exec_depends>
onrmw_implementation
I'm not sure about
<exec_depends>
. But<build_depend>
looks more reasonable to includermw
instead ofrmw_implementation
becausermw
is been used in the CMakeLists.txtAnother thing is that "rosidl_generator_c" is set as a
<build depend>
and<exec_depends>
but then in the CMakeLists.txt is not beenfind_package
Removed
rosidl_generator_c
dependency