Mark optional dependencies in package.xml. (#376)

personalrobotics · Apr 22, 2018 · 5cc94e7 · 5cc94e7
1 parent 7724899
commit 5cc94e7
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 12 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -34,6 +34,7 @@
 * Build & Testing & ETC
 
   * Fixed Eigen memory alignment issues on 32bit Ubuntu: [#368](https://github.com/personalrobotics/aikido/pull/368)
+  * Defined optional dependencies: [#376](https://github.com/personalrobotics/aikido/pull/376)
 
 ### 0.2.0 (2018-01-09)
 

diff --git a/package.xml b/package.xml
@@ -20,18 +20,18 @@
   <!-- Workaround to build DART with optional features enabled. -->
   <depend>nlopt</depend>
 
-  <!-- Optional dependencies. Uncomment these if you have them installed. -->
-  <!-- <depend>libmicrohttpd</depend> -->
-  <!-- <depend>roscpp</depend> -->
-  <!-- <depend>actionlib</depend> -->
-  <!-- <depend>control_msgs</depend> -->
-  <!-- <depend>geometry_msgs</depend> -->
-  <!-- <depend>interactive_markers</depend> -->
-  <!-- <depend>std_msgs</depend> -->
-  <!-- <depend>tf</depend> -->
-  <!-- <depend>trajectory_msgs</depend> -->
-  <!-- <depend>visualization_msgs</depend> -->
-  <!-- <depend>pr_control_msgs</depend> -->
+  <!-- Optional dependencies. Delete these if you don't have them. -->
+  <depend>libmicrohttpd</depend>
+  <depend>roscpp</depend>
+  <depend>actionlib</depend>
+  <depend>control_msgs</depend>
+  <depend>geometry_msgs</depend>
+  <depend>interactive_markers</depend>
+  <depend>std_msgs</depend>
+  <depend>tf</depend>
+  <depend>trajectory_msgs</depend>
+  <depend>visualization_msgs</depend>
+  <depend>pr_control_msgs</depend>
 
   <doc_depend>doxygen</doc_depend>
 
@@ -47,5 +47,18 @@
 
   <export>
     <build_type>cmake</build_type>
+
+    <!-- Optional dependencies. -->
+    <optional>libmicrohttpd</optional>
+    <optional>roscpp</optional>
+    <optional>actionlib</optional>
+    <optional>control_msgs</optional>
+    <optional>geometry_msgs</optional>
+    <optional>interactive_markers</optional>
+    <optional>std_msgs</optional>
+    <optional>tf</optional>
+    <optional>trajectory_msgs</optional>
+    <optional>visualization_msgs</optional>
+    <optional>pr_control_msgs</optional>
   </export>
 </package>