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

Add Espressif ESP-IDF template project #6682

Merged
merged 2 commits into from
Aug 15, 2023

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Aug 7, 2023

Description

Adds a new, minimally viable wolfSSL example project template for the Espressif ESP-IDF using an ESP32.

I'd like to include this in my upcoming Getting Started with wolfSSL on the Espressif ESP32 webinar.

Fixes zd# n/a

Testing

How did you test?

See included README.md

Checklist

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

See #6234 for a roadmap of Espressif updates.

@gojimmypi
Copy link
Contributor Author

@bandi13 I'm not sure if Jenkins will need any attention, but notice this example is a bit different from the others in where the expected user_settings.h file is located.

This PR has the user_settings.h in the project. The other current examples share the user_settings.h in the wolfSSL include directory.

The new method of pointing to wolfSSL source will be editing each respective project root CMakeLists.txt:

set(WOLFSSL_ROOT "~/workspace/wolfssl-other-source")

The examples here will know to find wolfSSL 7 parent directories up.

Perhaps it would be easiest for me to simply create a PR with the updated examples and see what happens.

@gojimmypi
Copy link
Contributor Author

retest this please

@gojimmypi gojimmypi requested review from JacobBarthelmeh and wolfSSL-Bot and removed request for dgarske and bandi13 August 15, 2023 19:12
@gojimmypi gojimmypi removed the request for review from wolfSSL-Bot August 15, 2023 21:17
@JacobBarthelmeh JacobBarthelmeh merged commit ec818e5 into wolfSSL:master Aug 15, 2023
73 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants