-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use moveit repo's travis config #47
Conversation
Building this together with latest MoveIt exhausts the time limit provided by Travis, particularly when running clang-tidy. I suggest building this repo against the MoveIt source container, which ensures that we check against the latest MoveIt source, but dramatically speed up the Travis check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #47 (comment).
@rhaschke i like that plan. can you make a PR to this PR, or just open another one? |
@davetcoleman I pushed a proposal to clean up .travis.yml. |
I pushed corrections myself, so dismissing my original review.
f75ed77
to
e96494f
Compare
This branch cannot be compiled on Kinetic (against kinetic-devel branch of MoveIt) anymore. It's only possible to build against melodic-devel or master of MoveIt. Do you really want to test this master branch against Kinetic as well (as indicated in #47 (comment))? |
It would be best if we still tested against Ubuntu 16.04, but no need to test against |
206be03
to
f200ca0
Compare
- For Melodic, use source container as ROS underlay to avoid building MoveIt. - For Kinetic, build MoveIt master branch too.
- use const references where possible - simply return of booleans
- order of member initialization - unused variables
@davetcoleman, I modified the Travis config to build MoveIt as well on 16.04. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good, thanks @rhaschke !
+1 |
* cleanup .travis.yml - For Melodic, use source container as ROS underlay to avoid building MoveIt. - For Kinetic, build MoveIt master branch too. * apply clang-tidy fixes - use const references where possible - simply return of booleans * use compile-time switches to disable/enable code blocks * clang-format * fix remaining compiler warnings - order of member initialization - unused variables
Currently travis is failing