Skip to content

Ignore build file warnings for Arduino; Update examples #7329

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

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Mar 13, 2024

Description

Defines WOLFSSL_IGNORE_FILE_WARN for the Arduino platform to ignore compile-time warnings in Arduino due to build system limitations noted in wolfSSL/Arduino-wolfSSL#1 with a root cause described in arduino/arduino-cli#631.

Both the wolfcrypt default settings.h and example Arduino user_settings.h file have been updated.

This is a nice-to-have, but not essential update to hopefully make it into the upcoming release.

Without this update, there are warnings such as this one observed in the PlatformIO Arduino build:

.pio/libdeps/esp32dev/wolfssl/src/wolfcrypt/src/bio.c:35:10: warning: #warning bio.c does not need to be compiled separately from ssl.c [-Wcpp]
         #warning bio.c does not need to be compiled separately from ssl.c
          ^~~~~~~

The Arduino warnings (and in fact the entire log) is hidden unless otherwise turned enabled for the Arduino IDE. PlatformIO is the opposite: showing the entire compiler log by default.

  • edit: I needed to refresh/rebase from upstream, so I took the opportunity to also polish existing Arduino examples as well as add a new, bare-bones wolfssl_version.ino example.

Fixes zd# n/a

Fixes #7330

Testing

How did you test?

Tested in Arduino only.

Checklist

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

@gojimmypi gojimmypi changed the title Ignore build file warnings for Arduino Ignore build file warnings for Arduino; Update examples Mar 13, 2024
@dgarske dgarske merged commit 924c0fd into wolfSSL:master Mar 13, 2024
@gojimmypi gojimmypi deleted the PR-Arduino-Build branch October 9, 2024 18:09
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.

Arduino Nano ESP32 examples fail to build in Arduino IDE
3 participants