lagadic / vision_visp

ViSP stack for ROS
http://wiki.ros.org/vision_visp
GNU General Public License v2.0
184 stars 90 forks source link

Missing zbar/libdmtx dependencies in visp_auto_tracker #17

Closed bchretien closed 10 years ago

bchretien commented 10 years ago

While packaging visp_auto_tracker for Arch Linux, I found missing dependencies. Indeed, the following lines can be found in package.xml:

<buildtool_depend>catkin</buildtool_depend>
<build_depend>visp_bridge</build_depend>
<build_depend>visp_tracker</build_depend>
<!-- to fix: add zbar and libdmtx-dev dependencies -->
<!-- do it manually using: sudo apt-get install libzbar-dev libdmtx-dev -->

Why was this just commented and not fixed? It seems that rosdep is up-to-date.

fspindle commented 10 years ago

I have tried to fix it few weeks ago but without success.

What I have done:

In package.xml add

zbar libdmtx-dev zbar libdmtx-dev

and in CMakeLists.txt add catkin_package( CATKIN_DEPENDS visp_bridge visp_tracker DEPENDS zbar libdmtx-dev )

$ rosdep check visp_auto_tracker System dependencies have not been satisified: apt libzbar-dev apt libdmtx-dev ERROR[visp_auto_tracker]: Cannot locate rosdep definition for [visp_tracker] rosdep key : visp_tracker OS name : ubuntu OS version : raring Data:

$ rosdep install visp_auto_tracker ERROR: the following packages/stacks could not have their rosdep keys resolved to system dependencies: visp_auto_tracker: Cannot locate rosdep definition for [visp_tracker]

Any idea would be appreciated

bchretien commented 10 years ago

I can take a look at this. There were some errors into catkin that were fixed a few weeks ago that influenced catkin_package.

bchretien commented 10 years ago

@fspindle: can you check this patch https://github.com/bchretien/vision_visp/commit/df9589a7e360f2fc93b29ad3455b45c99f5f4723?

bchretien commented 10 years ago

Apparently some of it is already solved in #19.

thomas-moulard commented 10 years ago

@bchretien can you tell me what still should be fixed? 0.7.1 should fix most of your issues.

thomas-moulard commented 10 years ago

Fixed in b8fbe4d81ed4c0e72879ecb3116db80a5886ff4f.

Closing, please re-open if necessary.