-
Notifications
You must be signed in to change notification settings - Fork 918
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
Include directories should be added as system include directories #924
Comments
This error seems unrelated to whether things are system includes or not. What compiler version are you using? |
@kilograham I'm using Fedora 36, this is the compiler identification according to CMake
I have created a minimal example on GitHub where you can see the full compiler output https://github.com/ooxi/pico-sdk-pedantic/runs/7435146674?check_suite_focus=true These are the compile settings for my source file which includes
|
The SDK will not compile with both |
@kilograham first of all, thank you very much for your fast reaction! Very appreciated :-)
I do not want to compile the SDK with Using I will come back with a proof of concept :-) |
@kilograham I have created a proof of concept which is sufficient to compile the example with I would be happy to clean up the patch and prepare a PR if you consider accepting it. |
Ah, ok i see the point... warnings are suppressed in system headers. |
@kilograham feel free to assign the issue to me, I will prepare a PR against develop |
duplicate of #82 |
oh this has a PR ;-) |
Currently all include directories are added as none system include directories. This causes warnings and errors when compiling my own source code
The text was updated successfully, but these errors were encountered: