Skip to content

Run Make ESP Update#400

Open
jlpoltrack wants to merge 1 commit intoolliw42:mainfrom
jlpoltrack:run-make-esp-update
Open

Run Make ESP Update#400
jlpoltrack wants to merge 1 commit intoolliw42:mainfrom
jlpoltrack:run-make-esp-update

Conversation

@jlpoltrack
Copy link
Collaborator

Bunch of changes:

  • Add support for Linux & Mac
  • Add parallel build support, compiles multiple targets about 30% faster
  • New list targets and flash arguments
  • Documentation for all arguments

Note - first compile can take some time while PIO is getting setup

@olliw42
Copy link
Owner

olliw42 commented Jan 25, 2026

hm
except of the Lin&Mac support, which makes sense, can you pl explain what issues this solves?
I note that it blows up the code from 190 lines to 600 lines, i.e., by more than a factor of THREE (!!!) just for what to me at first sight appears to not be really relevant for the pruposes of this script.

I guess I should mention that these tools came to life entrirely and purely as tools for not having to compile manually all these firmwares but to somewhat relieve the burden oif preparing releases. They were and are not intended as general purpose alternatives to e.g. VS.

@jlpoltrack
Copy link
Collaborator Author

can you pl explain what issues this solves?

  • For multiple targets, performance is better. Same logic could be applied to ST script as well, I measured something like 4.5x faster when applying multiprocessing there as targets are built sequentially one file at a time today.
  • Being able to do incremental builds and flash is super helpful for development without requiring one to use the VS Code IDE. Currently being locked to an IDE for ESP / ST development is annoying. One should be able to use one IDE and build / flash to both platforms.

I note that it blows up the code from 190 lines to 600 lines

Sure, but a lot of these are comments and redone formatting for the output. Output now looks like:

image

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