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
Mirror magni_robot in loki_robot #3
Conversation
This is the first step in resolving issue #1
<launch> | ||
<arg name="gui" default="False" /> | ||
<param name="use_gui" value="$(arg gui)"/> | ||
<param name="robot_description" textfile="$(find magni_description)/urdf/loki.urdf" /> |
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.
Seems to reference magni_description directory
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.
Fixed, also greped for magni and removed all references to it.
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.
Looks OK to me, but I don't have the hardware to test on.
you need to use exec_depend on all the subpackages in the metapackage otherwise the installing the metapackage won't pull in the others
Fixes #1
This should significantly simplify the whole loki setup situation from the mess it is right now.
It still isn't ideal because of all the code duplication, but we can work on fixing that.
Please look for any errors, as I probably screwed up somewhere, and I haven't tested this on anything.