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

Introduce Espressif common cmake #8535

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gojimmypi
Copy link
Contributor

Description

Introduces a new wolfSSL Espressif CMake Library location: IDE/Espressif/ESP-IDF/cmake.

This directory contains common functions used in various examples and components.

Instead of duplicating and maintaining functions in various examples and components, this common library can be used.

Although it can be used as-is, the intent is to include it in the published wolfSSL Managed Component and include it from there when possible.

Inspiration came from my most recent publish of the Staging Component for wolfMQTT, in particular the examples. Note there are more examples for the wolfssl Managed Component and examples for the wolfssh Managed Component - all of which need to check for duplicate components.

The initial file contains just one commonly needed function to check for duplicate component directories:

CHECK_DUPLICATE_LIBRARIES

Searches for duplicate directories containing duplicate component libraries.

Parameters:

  • RESULT_VAR (output variable)
  • KEYWORD (e.g. "wolfssl", "wolfmqtt", etc)

Fixes zd# n/a

Testing

Tested only when directly implemented in other CMakeFiles.txt. This include capability has otherwise NOT yet been implemented in any shared manner, pending review and acceptance of this PR.

Checklist

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

@gojimmypi
Copy link
Contributor Author

Jenkins retest this please.

To retry unrelated "java.io.StreamCorruptedException: invalid stream header: 636F7272"

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.

3 participants