gazebosim / gz-transport

Transport library for component communication based on publication/subscription and service calls.
https://gazebosim.org
Apache License 2.0
29 stars 42 forks source link

Use relative install path for gz tool data #492

Closed azeey closed 6 months ago

azeey commented 6 months ago

🦟 Bug fix

Summary

Fixes an error when building https://github.com/gazebo-release/gz_transport_vendor/ in the ROS buildfarm.

Similar to https://github.com/gazebosim/gz-tools/pull/137

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.73%. Comparing base (eac2e69) to head (ddc0444). Report is 11 commits behind head on gz-transport13.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## gz-transport13 #492 +/- ## ================================================== + Coverage 87.69% 87.73% +0.04% ================================================== Files 59 59 Lines 5704 5708 +4 ================================================== + Hits 5002 5008 +6 + Misses 702 700 -2 ```

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

azeey commented 6 months ago

The homebrew failure is due to a recent change in protobuf and not related to this PR. I'll go ahead and merge.