spot_ros icon indicating copy to clipboard operation
spot_ros copied to clipboard

Instructions for Using `cakin build` for Python3 Work-Around on Melodic

Open MarioGini opened this issue 3 years ago • 5 comments

As the description says, the build instructions suggest to use catkin_make when using the python3 workaround for ROS Melodic. I think that in general, using catkin build is preferred over catkin_make so maybe that section could suggest to e.g. add the additional CMake arguments to a caktin config profile using catkin config --cmake-args ...

MarioGini avatar Sep 09 '21 11:09 MarioGini

Realistically you can use either tool, and both will do the job of building the packages. We tend to use catkin_make in a lot of our examples as that command is automatically included (and is also what the ROS wiki uses in its tutorials).

I don't know that there would be a lot of benefit to adding the catkin build instructions when the provided instructions using catkin_make do work on Melodic. Melodic is in an awkward situation, since it exists after the Python3 EOL date, but is still stuck without official Python3 support and is stuck requiring these sorts of work-arounds.

civerachb-cpr avatar Sep 09 '21 15:09 civerachb-cpr

catkin_make is the official Melodic tool for building packages. catkin build is an alternative implementation. I think it's good the tutorials stick with the officially distributed build system.

peci1 avatar Sep 13 '21 03:09 peci1

I've edited the title of this issue. Anyone preferring to use catkin build can find this issue and use the instructions you've provided here, rather than modifying our core instructions to use a secondary build system.

civerachb-cpr avatar Sep 13 '21 13:09 civerachb-cpr

/home/agx3/ros2/devel/include/spot_msgs/LeaseArray.h:18:10: fatal error: spot_msgs/LeaseResource.h: No such file or directory #include <spot_msgs/LeaseResource.h> ^~~~~~~~~~~~~~~~~~~~~~~~~~~

BADAL244 avatar Oct 28 '21 11:10 BADAL244

This driver is for ROS1 (specifically Melodic and Noetic). The path to the file seems to indicate you're trying to use the driver for ROS2?

civerachb-cpr avatar Oct 29 '21 17:10 civerachb-cpr