Remove gtest from the sources.

ros / urdfdom

URDF parser

http://ros.org/wiki/urdf

Other

96 stars 132 forks source link

Closed clalancette closed 3 years ago

clalancette commented 3 years ago

It is only used very sparingly, and gets rid of a vendored package that causes warnings.

Signed-off-by: Chris Lalancette clalancette@openrobotics.org

@scpeters @sloretz I'd appreciate review of this, as it will get rid of one of the long-standing warnings in ROS 2 CI: https://ci.ros2.org/view/nightly/job/nightly_linux_clang_libcxx/750/clang/new/ .

Note that the other way we could go here is to update gtest to a newer version, but this seemed simpler to me.

clalancette commented 3 years ago

CI:

scpeters commented 3 years ago

feel free to merge

Karsten1987 commented 3 years ago

@clalancette can you please backport this to Foxy? Or at least the part indicated in #151

clalancette commented 3 years ago

See #155