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 Espressif make and cmake for ESP8266 and ESP32 series #8402

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Jan 31, 2025

Description

This PR updates the make and cmake build process for the Espressif examples.

  • VisualGDB is detected in make. Fixed builds for ESP8266 for VisualGDB.
  • Improved exclusion of select obj files during make builds for ESP8266.
  • Proper exclusion of driver and timer components in ESP8266 build.
  • Proper inclusion of pthreads component for ESP8266 CMake build.
  • Added support for template example make builds for ESP8266.
  • Improved watchdog disable on some ESP-IDF versions for RISC-V chipsets.
  • Fixed watchdog disable macro for ESP8266 on SDK 3.5.
  • Improved PROTOCOL_EXAMPLES_DIR detection.
  • Expanded Apple Home kit support in Kconfig.
  • Enable AES and AES-CTR for all examples. Warn if AES explicitly disabled.
  • General cleanup, spelling, example consistency.
  • Add missing ESP8266 VisualGDB wolfssl server example project file.
  • Improved ets_sys path / version detection. Thanks @5p4k see Fix ESP ROM header path for IDF >= 5.4 #8380
  • Update .gitignore to exclude more Visual Studio / VisualGDB / log files.
  • Remove references to Espressif aes.h.

Tested on ESP8266 v3.5 SDK, ESP-IDF v4.4.3 (on supported chips), ESP-IDF v5.2, and latest upstream master (v5.5)

edit: also fixes #8266

Fixes zd#

Testing

Tested on my Espressif jig

image

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

Sorry, something went wrong.

@gojimmypi gojimmypi requested a review from dgarske January 31, 2025 05:04
@dgarske dgarske removed their assignment Jan 31, 2025
@dgarske dgarske removed their request for review January 31, 2025 20:30
@gojimmypi gojimmypi marked this pull request as draft February 3, 2025 23:02
@gojimmypi
Copy link
Contributor Author

Setting to draft while I add the wolfSSH changes for wolfSSL/wolfssh-examples#8 (comment)

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please

@gojimmypi gojimmypi marked this pull request as ready for review February 4, 2025 03:47
@gojimmypi
Copy link
Contributor Author

This PR includes contents of #8418, please merge that one first, please.

@dgarske dgarske requested review from dgarske and removed request for JacobBarthelmeh February 4, 2025 21:30
@dgarske dgarske merged commit 743655b into wolfSSL:master Feb 4, 2025
163 checks passed
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.

[Bug]: Missing "esp32/rom/aes.h"
3 participants