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

Arduino Core 3 #407

Merged
merged 18 commits into from
Jul 5, 2024
Merged

Arduino Core 3 #407

merged 18 commits into from
Jul 5, 2024

Conversation

iranl
Copy link
Collaborator

@iranl iranl commented Jun 20, 2024

Description:

Most important changes

  • Based on Tasmota PlatformIO platform 2024.06.11 (Arduino Core 3.0.2 and ESP-IDF 5.1.4)
  • Build using Arduino as a (ESP-IDF) component
  • Uses esp-nimble-cpp instead of NimBLE-Arduino (which will probably not be supported with Arduino core 3.0 by the creator of both libraries (h2zero) going forward)
  • Change partition scheme to a big-little configuration. Requires one time flashing using USB/serial.
  • Unified source for the main binary and updater binary using if/else and defines
  • Add fully automatic HTTPS updating using the latest binaries from GitHub
  • Allow starting HTTPS updates using MQTT and Home Assistant
  • Add experimental support for ESP32-C6
  • Drop CMake support (at least as a guided option using VM/Docker/README)

Note that this PR comprises much more than just the step to Arduino Core 3 and ESP-IDF 5.1.4 (which would be enough of a change to warrant not only it's own PR but even it's own release).

Reason for all the other changes is:

  • The move to Arduino Core 3 required more flash
  • Which required the big-little setup
  • Which required the new updater
  • Which required the HTTPS updater (to prevent changing partition scheme again in the near future because the partion size for the little binary was too small)

To-Do

Related issue (if applicable): fixes #226, fixes #408, fixes #403, fixes #414

Checklist:

  • The pull request is done against the latest master branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works
  • I accept the CLA.

@iranl iranl added the enhancement New feature or request label Jun 20, 2024
@iranl iranl added this to the Arduino Core 3.0 milestone Jun 20, 2024
@iranl iranl requested a review from technyon June 20, 2024 17:46
@iranl
Copy link
Collaborator Author

iranl commented Jun 20, 2024

@technyon: Can you merge PR technyon/nuki_ble#3?

@technyon
Copy link
Owner

@technyon: Can you merge PR technyon/nuki_ble#3?

Unfortunately there are conflicts that have to be resolved first.

@iranl
Copy link
Collaborator Author

iranl commented Jun 22, 2024

@technyon: Can you merge PR technyon/nuki_ble#3?

Unfortunately there are conflicts that have to be resolved first.

Don't see any conflicts:

image

@technyon technyon merged commit 6b0100f into technyon:master Jul 5, 2024
1 of 11 checks passed
@iranl iranl mentioned this pull request Jul 5, 2024
5 tasks
@OnFreund
Copy link

OnFreund commented Aug 6, 2024

Requires one time flashing using USB/serial.

Will this overwrite configuration?

@iranl
Copy link
Collaborator Author

iranl commented Aug 6, 2024

No

@OnFreund
Copy link

OnFreund commented Aug 9, 2024

It did overwrite configuration...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants