No CATKIN_ENABLE_TESTING check

DFKI-NI / rospy_message_converter

Converts between Python dictionaries and JSON to ROS messages.

BSD 3-Clause "New" or "Revised" License

226 stars 101 forks source link

No CATKIN_ENABLE_TESTING check #19

Closed o-fugi closed 5 years ago

o-fugi commented 6 years ago

There should be an if(CATKIN_ENABLE_TESTING) around the add_rostest line in CMakeLists.txt. Otherwise, it throws an error when running catkin_make_isolated: The CMake code should only use it inside a conditional block which checks that testing is enabled.

Fix:

if(CATKIN_ENABLE_TESTING) add_rostest(...) endif()

mintar commented 5 years ago

Fixed in #9.