search

moveit / srdfdom

Semantic Robot Description Format
BSD 3-Clause "New" or "Revised" License
12 stars 68 forks source link

Travis for noetic #57

Closed v4hn closed 4 years ago

v4hn commented 4 years ago

Currently, this fails. But if somebody adjusts the moveit_ci infrastructure, here is the patch to test it already.

rhaschke commented 4 years ago

Do we really need a separate branch for noetic already? Let's try to postpone creation of new branches a long as possible. Otherwise we end up in a cherry-pick mess as in the main MoveIt repo.

ahcorde commented 4 years ago

@ros-pull-request-builder retest this please

ahcorde commented 4 years ago

CI is returning a Warning http://build.ros.org/job/Npr__srdfdom__ubuntu_focal_amd64/1/

08:51:46 CMake Warning at CMakeLists.txt:61 (message):
08:51:46   Locale nl_NL not available.  Locale test will not be meaningful.

https://github.com/ros-planning/srdfdom/blob/0232a1691a68fee4e90ad12ea6751175c8d1ad6b/CMakeLists.txt#L57-L62

Any thoughts?

rhaschke commented 4 years ago

This test requires the NL locale to be installed, which we do in our Travis CI. This is all fine.

rhaschke commented 4 years ago

Trigger CI.