RethinkRobotics / sawyer_simulator

Gazebo Simulation interface for the Sawyer Robot
http://sdk.rethinkrobotics.com/intera/Gazebo_Tutorial
Apache License 2.0
51 stars 64 forks source link

intera_core_msgs dependency issue in sawyer_sim_controllers #29

Closed jarvisschultz closed 6 years ago

jarvisschultz commented 6 years ago

When following directions on Gazebo Tutorial I'm greeted with the following error:

[  1%] Building CXX object sawyer_simulator/sawyer_gazebo/CMakeFiles/sawyer_robot_hw_sim.dir/src/sawyer_robot_hw_sim.cpp.o
In file included from /home/jarvis/52sawyerws/src/sawyer_simulator/sawyer_sim_controllers/src/sawyer_velocity_controller.cpp:17:0:
/home/jarvis/52sawyerws/src/sawyer_simulator/sawyer_sim_controllers/include/sawyer_sim_controllers/sawyer_velocity_controller.h:21:43: fatal error: intera_core_msgs/JointCommand.h: No such file or directory
compilation terminated.
sawyer_simulator/sawyer_sim_controllers/CMakeFiles/sawyer_sim_controllers.dir/build.make:86: recipe for target 'sawyer_simulator/sawyer_sim_controllers/CMakeFiles/sawyer_sim_controllers.dir/src/sawyer_velocity_co
ntroller.cpp.o' failed
make[2]: *** [sawyer_simulator/sawyer_sim_controllers/CMakeFiles/sawyer_sim_controllers.dir/src/sawyer_velocity_controller.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
Scanning dependencies of target intera_examples_gen

This indicates catkin is attempting to build the sawyer_robot_hw_sim.cpp.o target before the intera_core_msgs/JointCommand.msg target has been built. Likely need to explicitly add a dependency on ${catkin_EXPORTED_TARGETS} for this lib.

IanTheEngineer commented 6 years ago

Right you are @jarvisschultz. Thank you for bringing this to my attention. I had been using catkin build from catkin-tools which wouldn't allow this situation to happen, by resolving the dependencies first and then building. However, the catkin_make needs to work as well. It seems there are a few other missing lines from the CMakeLists.txt in this repo, so I'll patch them up and tag you in the Pull Request. Thanks!

IanTheEngineer commented 6 years ago

Resolved by #30 and #31