Added rosdep key for ignition-edifice, made it a mandatory dependency (…

…#31) Signed-off-by: Luca Della Vedova <[email protected]>
open-rmf · Aug 18, 2021 · 7e981e4 · 7e981e4
1 parent a80cb6e
commit 7e981e4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/rmf_building_sim_ignition_plugins/CMakeLists.txt b/rmf_building_sim_ignition_plugins/CMakeLists.txt
@@ -26,13 +26,8 @@ if (NOT ignition-cmake2_FOUND)
   message("ignition-cmake not found, skipping ignition gazebo plugins")
   return()
 endif()
-ign_find_package(ignition-gazebo5 QUIET)
-if (NOT ignition-gazebo5_FOUND)
-  message("Wrong ignition-gazebo version found. Plugins require ignition-gazebo5 (edifice)")
-  # To generate a setup.bash and avoid CMake warnings
-  ament_package()
-  return()
-endif()
+
+ign_find_package(ignition-gazebo5 REQUIRED)
 set(IGN_GAZEBO_VER 5)
 ign_find_package(ignition-plugin1 REQUIRED COMPONENTS register)
 set(IGN_PLUGIN_VER 1)

diff --git a/rmf_building_sim_ignition_plugins/package.xml b/rmf_building_sim_ignition_plugins/package.xml
@@ -19,6 +19,7 @@
   <depend>libqt5-qml</depend>
   <depend>libqt5-quick</depend>
   <depend>menge_vendor</depend>
+  <depend>ignition-edifice</depend>
 
   <export>
     <build_type>ament_cmake</build_type>