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

Compiler error #6523

Merged
merged 1 commit into from
Apr 4, 2022
Merged

Compiler error #6523

merged 1 commit into from
Apr 4, 2022

Conversation

nimaltd
Copy link
Contributor

@nimaltd nimaltd commented Apr 3, 2022

Error on compile Arduino as an ESP-IDF component

By completing this PR sufficiently, you help us to improve the quality of Release Notes

Checklist

  1. [x ] Please provide specific title of the PR describing the change, including the component name (eg. „Update of Documentation link on Readme.md“)
  2. [ x] Please provide related links (eg. Issue, other Project, submodule PR..)
  3. [ x] Please check Contributing guide

This entire section above can be deleted if all items are checked.


Summary

coude not compile on esp-idf

Impact

add #ifndef and #endif

Related links

Error on compile Arduino as an ESP-IDF component
@Jason2866
Copy link
Collaborator

Could not reproduce with actual (master) Arduino version and supported IDF release/v4.4 commit c29343eb94

@github-actions
Copy link
Contributor

github-actions bot commented Apr 4, 2022

Unit Test Results

0 files  0 suites   0s ⏱️
0 tests 0 ✔️ 0 💤 0

Results for commit b0c477f.

@me-no-dev me-no-dev merged commit 5d6b9d0 into espressif:master Apr 4, 2022
@me-no-dev
Copy link
Member

Thanks @nimaltd :) it seems we have missed the guards in this one :)

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.

None yet

3 participants