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

Equalize include flag and warning flags with platform.txt #44

Open
maxgerhardt opened this issue Sep 25, 2021 · 0 comments
Open

Equalize include flag and warning flags with platform.txt #44

maxgerhardt opened this issue Sep 25, 2021 · 0 comments

Comments

@maxgerhardt
Copy link
Member

PR #43 changed some -I includes to -isystem includes so that we don't get warnings from the SDK and ArduinoCore-API code that we don't touch.

Further, the Arduino core builds with warning flags (-Wall -Wextra) that PlatformIO doesn't use.

All these flags should be equalized.

A possible caveat with changing -I to -isystem flags in the PlatformIO system is however that when moving folders from the CPPPATH to CCFLAGS (-isystem flags), they might disappear from the include search directory in generated IntelliSense configuration files (c_cpp_properties.json). This would greatly limit IDE usefullness when using PlatformIO. If that is the case, an issue in the PlatformIO core must be opened (or accepted that we can't change -I to -isystem comfortably yet).

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

No branches or pull requests

1 participant