segbot_sensors: missing dependency on depthimage_to_laserscan #20
Assignees
Labels
Comments
ghost
assigned
jack-oquin
added a commit
that referenced
this issue
Sep 19, 2013
jack-oquin
added a commit
that referenced
this issue
Sep 19, 2013
|
Added the unit tests, but had to comment out this one due to ros-drivers/openni_launch#10: |
jack-oquin
added a commit
that referenced
this issue
Sep 19, 2013
jack-oquin
added a commit
that referenced
this issue
Sep 19, 2013
|
I checked again, and the fix for ros-drivers/openni_launch#10 has been merged but still not released. Holding this issue open until we can remove the work-around for that bug. |
jack-oquin
added a commit
that referenced
this issue
Apr 20, 2014
|
Closing this issue now that the openni_launch fix has been released and full roslaunch checking is now enabled here. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed this was missing today when I installed the software on my home development machine using the Installing our code-base on a machine instructions from the wiki.
As a quick fix, I added ros-hydro-depthimage-to-laserscan to the list of APT packages to install.
But, I think a better fix is to add the proper
<run-depend>to segbot_sensors.The best fix is to also add unit test coverage verifying that all packages required by a set of launch files are mentioned as dependencies.
The text was updated successfully, but these errors were encountered: