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

Improve CI build #71

Closed
wants to merge 34 commits into from
Closed

Improve CI build #71

wants to merge 34 commits into from

Conversation

BorjaOuterelo
Copy link
Contributor

Uses a new base image: microros/ci:dashing to build firmware clients.
This base image has the toolchains pre-downloaded it should avoid the spurious download error.

@iluetkeb
Copy link
Contributor

Looks like it's not finding the gcc?

@BorjaOuterelo
Copy link
Contributor Author

Looks like it's not finding the gcc?

True. Some path issues. I will push some changes.

@pablogs9 pablogs9 self-assigned this Feb 25, 2020
@pablogs9
Copy link
Member

TODO: add flashing tool check to CI in order to check if e.g. dfu-utils dependency is resolved

@pablogs9
Copy link
Member

pablogs9 commented Mar 5, 2020

@jfm92 are you aware of this CI? I'm removing my assignment and assigning you

And maybe is needed to check this PR https://github.com/micro-ROS/olimex_e407_extensions/pull/2 beacuse there was some issues with downloading the whole toolchain (~800 MB) in the dockers for some checks in the CI

@pablogs9 pablogs9 assigned jfm92 and unassigned pablogs9 Mar 5, 2020
@julionce
Copy link

This PR was addressed at #86.

@julionce julionce closed this Mar 18, 2020
@julionce julionce deleted the feature/improve_ci branch March 19, 2020 07:05
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

Successfully merging this pull request may close these issues.

5 participants