Revive the survive_in_catkin_make build property, for the ability to use catkin_tools to build survive_publisher. Property was originally added in https://github.com/cntools/libsurvive/commit/42bfce9e1fc54a2ab53bf15ae835882f449a60d8, but functionality was lost during CMakeLists.txt refactor, whether intentional or not. With this change, libsurvive can be put in a catkin workspace, and be built simply by running catkin build. Without this change, the CMakeLists.txt will have a circular inclusion and fail to build.
Also, catkin_package() must be called before any targets are added with add_executable, so that has been moved further up in the CMakeLists.txt. Quoting ROS Wiki catkin - 6. catkin_package - "This function (catkin_package) must be called before declaring any targets with add_library() or add_executable()". Currently, with catkin_package being called after add_executable, the executables aren't built into the catkin devel/ space as expected.
Revive the
survive_in_catkin_make
build property, for the ability to use catkin_tools to build survive_publisher. Property was originally added in https://github.com/cntools/libsurvive/commit/42bfce9e1fc54a2ab53bf15ae835882f449a60d8, but functionality was lost during CMakeLists.txt refactor, whether intentional or not. With this change, libsurvive can be put in a catkin workspace, and be built simply by runningcatkin build
. Without this change, the CMakeLists.txt will have a circular inclusion and fail to build.Also,
catkin_package()
must be called before any targets are added withadd_executable
, so that has been moved further up in theCMakeLists.txt
. Quoting ROS Wiki catkin - 6. catkin_package - "This function (catkin_package) must be called before declaring any targets with add_library() or add_executable()". Currently, with catkin_package being called after add_executable, the executables aren't built into the catkindevel/
space as expected.