Add ROS 2 condition for yaml_cpp_vendor package

Prevent attempting to rosdep install a nonexistent package in ROS Noetic. Also update docker pipeline to trigger on changes to package.xml files across the repository. Signed-off-by: Nahuel Espinosa <[email protected]>
Ekumen-OS · May 28, 2024 · 135cd6b · 135cd6b
1 parent 0739c5e
commit 135cd6b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/.github/workflows/docker_ci_pipeline.yml b/.github/workflows/docker_ci_pipeline.yml
@@ -3,10 +3,14 @@ name: Docker CI pipeline
 on:
   push:
     branches: [ main ]
-    paths: 'docker/**'
+    paths:
+      - 'docker/**'
+      - '**/package.xml'
   pull_request:
     branches: [ main ]
-    paths: 'docker/**'
+    paths:
+      - 'docker/**'
+      - '**/package.xml'
   schedule:
     - cron: '0 0 * * 0'  # Weekly on Sunday
 

diff --git a/beluga_tutorial/package.xml b/beluga_tutorial/package.xml
@@ -16,7 +16,7 @@
   <buildtool_depend>cmake</buildtool_depend>
 
   <depend>beluga</depend>
-  <depend>yaml_cpp_vendor</depend>
+  <depend condition="$ROS_VERSION == 2">yaml_cpp_vendor</depend>
 
   <export>
     <build_type>cmake</build_type>