Merge pull request #13 from NikolausDemmel/patch1

Some cleanup in package.xml and CMakeLists.txt
ros-perception · Jul 19, 2014 · ce6a136 · ce6a136
2 parents 91a3959 + 16fe2a0
commit ce6a136
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/theora_image_transport/CMakeLists.txt b/theora_image_transport/CMakeLists.txt
@@ -2,7 +2,7 @@ cmake_minimum_required(VERSION 2.8.3)
 project(theora_image_transport)
 
 find_package(OpenCV REQUIRED)
-find_package(catkin REQUIRED cv_bridge dynamic_reconfigure image_transport message_generation tf rosbag)
+find_package(catkin REQUIRED COMPONENTS cv_bridge dynamic_reconfigure image_transport message_generation tf rosbag pluginlib std_msgs)
 
 add_message_files(DIRECTORY msg FILES Packet.msg)
 
@@ -21,7 +21,7 @@ generate_dynamic_reconfigure_options(cfg/TheoraPublisher.cfg cfg/TheoraSubscribe
 catkin_package(
   INCLUDE_DIRS include
   LIBRARIES ${PROJECT_NAME}
-  CATKIN_DEPENDS message_runtime
+  CATKIN_DEPENDS message_runtime std_msgs
 )
 
 include_directories(include ${catkin_INCLUDE_DIRS})
@@ -43,20 +43,18 @@ add_definitions(${PC_OGG_CFLAGS_OTHER}
 )
 
 add_library(${PROJECT_NAME} src/theora_publisher.cpp src/theora_subscriber.cpp src/manifest.cpp)
-add_dependencies(${PROJECT_NAME} ${PROJECT_NAME}_gencfg)
+add_dependencies(${PROJECT_NAME} ${PROJECT_NAME}_gencfg ${PROJECT_NAME}_gencpp)
 target_link_libraries(${PROJECT_NAME} ${catkin_LIBRARIES}
                                       ${OpenCV_LIBRARIES}
                                       ${PC_OGG_LIBRARIES}
                                       ${PC_THEORA_LIBRARIES}
                                       ${PC_THEORAENC_LIBRARIES}
                                       ${PC_THEORADEC_LIBRARIES}
-#                                      ogg theora theoraenc theoradec
 )
 
 class_loader_hide_library_symbols(${PROJECT_NAME})
 
 add_executable(ogg_saver src/ogg_saver.cpp)
-include_directories(ogg_saver include cfg/cpp)
 target_link_libraries(ogg_saver ${PC_THEORA_LIBRARY} 
                                 ${PC_OGG_LIBRARY} 
                                 ${OpenCV_LIBRARIES} 

diff --git a/theora_image_transport/package.xml b/theora_image_transport/package.xml
@@ -23,6 +23,7 @@
   <build_depend>pluginlib</build_depend>
   <build_depend>rosbag</build_depend>
   <build_depend>tf</build_depend>
+  <build_depend>std_msgs</build_depend>
 
   <run_depend>cv_bridge</run_depend>
   <run_depend>dynamic_reconfigure</run_depend>
@@ -33,9 +34,9 @@
   <run_depend>pluginlib</run_depend>
   <run_depend>rosbag</run_depend>
   <run_depend>tf</run_depend>
+  <run_depend>std_msgs</run_depend>
 
   <export>
-    <rviz plugin="${prefix}/plugin_description.xml"/>
     <image_transport plugin="${prefix}/theora_plugins.xml" />
   </export>
 </package>