Skip to content
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

[travis] Check for missing dependencies #24

Open
fmessmer opened this issue Apr 2, 2016 · 7 comments
Open

[travis] Check for missing dependencies #24

fmessmer opened this issue Apr 2, 2016 · 7 comments

Comments

@fmessmer
Copy link
Member

fmessmer commented Apr 2, 2016

@ipa-fmw @ipa-mdl

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

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

@fmessmer FYI

@mathias-luedtke
Copy link

roslint checks things like that..

@fmessmer
Copy link
Member Author

fmessmer commented Apr 5, 2016

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
Copy link

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
Copy link

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

@fmessmer
Copy link
Member Author

fmessmer commented Apr 5, 2016

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

@mathias-luedtke
Copy link

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
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants