[travis] Check for missing dependencies - Githubissues

ipa320 / care-o-bot

http://www.care-o-bot.de

Apache License 2.0

48 stars 41 forks source link

[travis] Check for missing dependencies #24

Open fmessmer opened 8 years ago

fmessmer commented 8 years ago

@ipa-fmw @ipa-mdl

Is there a way to configure travis in a way so that it tells us about missing dependencies as in

cob_bms_driver: missing dependency to diagnostic_updater
cob_sick_lms1xx: missing dependency to diagnostic_msgs

We would need something like "rosdep install isolated/packagewise-with-uninstall"?

@fmessmer FYI

mathias-luedtke commented 8 years ago

roslint checks things like that..

fmessmer commented 8 years ago

Is it possible to add this to travis? How? Do you have an example? Or would it be an additional (automatic) PR/CI test?

mathias-luedtke commented 8 years ago

I don't have much experience with roslint so far. But if it returns a non-zero exit code on failures you can add the invocation to https://github.com/ipa320/care-o-bot/blob/indigo_dev/.travis.script.sh#L17.

mathias-luedtke commented 8 years ago

I actually did not meant roslint, there was this other tool, I forgot the name of.

fmessmer commented 8 years ago

roscompile: https://github.com/DLu/roscompile

mathias-luedtke commented 8 years ago

Ah yes! We could add a roscompile.yaml to our travis suite and copy it over to the correct location. I haven't testes the return codes yet. If it does not return on failure, we should file an issue.

mathias-luedtke commented 8 years ago

https://github.com/DLu/roscompile/blob/master/scripts/roscompile#L9 :-/