Skip to content

Update dockerfile baseimage#87

Merged
pablogs9 merged 2 commits intomicro-ROS:galacticfrom
gustavo-oliveira-mendonca:update-dockerfile-baseimage
Aug 26, 2021
Merged

Update dockerfile baseimage#87
pablogs9 merged 2 commits intomicro-ROS:galacticfrom
gustavo-oliveira-mendonca:update-dockerfile-baseimage

Conversation

@gustavo-oliveira-mendonca
Copy link
Copy Markdown
Contributor

I couldn't correctly configure the esp-idf toolchain on my machine and I was very happy to know that there was the possibility of using Docker for the build, upload and monitor processes. I wasn't successful with the current Dockerfile, but updating the base image to version 4.3 of the toolchain and adding a missing dependency fixed the issues.

@pablogs9
Copy link
Copy Markdown
Member

@mergify backport main

@pablogs9
Copy link
Copy Markdown
Member

@gustavo-oliveira-mendonca Thanks a lot for the contribution!

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Aug 26, 2021

Command backport main: pending

Waiting for the pull request to get merged

Hey, I reacted but my real name is @Mergifyio

@pablogs9 pablogs9 merged commit ac1ede4 into micro-ROS:galactic Aug 26, 2021
mergify Bot pushed a commit that referenced this pull request Aug 26, 2021
* Update baseimage to espressif/idf:release-v4.3

* Adding missing python dep importlib-resources

Co-authored-by: Gustavo O. Mendonça <gustavo.mendonca@ihm.com.br>
(cherry picked from commit ac1ede4)
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Aug 26, 2021

Command backport main: success

Backports have been created

pablogs9 pushed a commit that referenced this pull request Aug 26, 2021
* Update baseimage to espressif/idf:release-v4.3

* Adding missing python dep importlib-resources

Co-authored-by: Gustavo O. Mendonça <gustavo.mendonca@ihm.com.br>
(cherry picked from commit ac1ede4)

Co-authored-by: Gustavo Oliveira Mendonça <gustavo36@gmail.com>
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.

2 participants