colcon / colcon-ros

Extension for colcon to support ROS packages
http://colcon.readthedocs.io
Apache License 2.0
13 stars 26 forks source link

Also catch FileNotFoundError when detecting multiarch #159

Closed cottsay closed 3 months ago

cottsay commented 3 months ago

When the executable we're trying to invoke doesn't exist at all, we'll see a FileNotFoundError and not a subprocess.CalledProcessError.

Fixes #130

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 6.92%. Comparing base (c5452db) to head (a2d7b1c). Report is 1 commits behind head on master.

Files Patch % Lines
colcon_ros/task/cmake/__init__.py 0.00% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #159 +/- ## ====================================== Coverage 6.92% 6.92% ====================================== Files 20 20 Lines 650 650 Branches 139 139 ====================================== Hits 45 45 Misses 605 605 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.