-
-
Notifications
You must be signed in to change notification settings - Fork 176
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 Dockerfiles #2825
Improve Dockerfiles #2825
Conversation
Superficial changes.
NOTE: I would like to question whether AND: whether the hack for Python 3.10 is still required (or use |
Yes. That's the latest version in the IDF version we're using.
|
Fix issue with gcc location
That I really don't remember why... |
Fix duplicate clone
Use Python image rather than custom repo
Runs on PR's
Comment out faulty line for test
Fix spacing?
Fix sed command
Use submodules
Fix extra slash
Include TI target as last container.
Comment out TI as known broken
Use python:3.10 container
Improve for all containers and platforms.
Fix matrix vars
@Ellerbach @josesimoes I hope that this PR is acceptable. It is a little wider in range than I hoped and given issues I had recently (as shown by the workflow runs), decided it was time to add a smoke test. It will help before #2816 is merged, as it adds multiple improvements to ensure builds and tests against them. (TI currently commented out as broken (and known see https://discord.com/channels/478725473862549535/1173310502202511450)). |
Add matrix TI All (commented out)
Git SVN is just too slow.
Improves maintainability.
Fixes for All when choosing lwIP
Rebumps containers
(outdated lwip).
Linked to #2832 |
Should be kept seperate.
Description
hex2dfu
rather than tagged version.Motivation and Context
How Has This Been Tested?
New (smoketest) workflow.
Screenshots
Types of changes
Checklist