Added testsuite #27
Added testsuite #27
Conversation
|
Tests compile but are never run from rostest, closing to address it. Will reopen once the tests actually run. |
…o support calling generate_dynamic_reconfigure_options from dynamic_reconfigure's source.
|
This branch now contains a simple test that runs with |
| catkin_package(LIBRARIES dynamic_reconfigure_config_init_mutex | ||
| INCLUDE_DIRS include | ||
| CATKIN_DEPENDS message_runtime std_msgs | ||
| CFG_EXTRAS extras.cmake | ||
| CFG_EXTRAS dynamic_reconfigure-macros.cmake dynamic_reconfigure-extras.cmake |
There was a problem hiding this comment.
You should only need dynamic_reconfigure-extras.cmake.em since that already includes the other file.
|
@dirk-thomas I removed the extra cmake module. I had to add roscpp_LIBRARIES explicitly to make this work on Hydro, which I don't really understand why. Shouldn't catkin_LIBRARIES contain roscpp_serialization and roscpp after find_package(catkin ...)? |
…est server. Added a one second delay to make sure that the configuration changes are committed on the server
|
The |
|
@dirk-thomas Thanks, just updated package.xml and CMakeLists.txt. Is that version of catkin getting backported to Groovy as well? |
|
Yes |
This branch reenables tests and updates the testsuite to be compiled on Hydro and onwards.