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

[nrf fromtree] cmake: use the warnings_as_errors flag for cpp files #2350

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

Conversation

tejlmand
Copy link
Contributor

@tejlmand tejlmand commented Dec 9, 2024

Next Zephyr upmerge will enable warnings-as-errors for C++.
This commit is cherry-picked to allow early fix of C++ warnings in NCS code base.


C++ file compilation is actually missing the warning-as-error handling, causing warnings in build files to be unnoticed in CI. Add a flag to handle them as well.

Signed-off-by: Fabio Baltieri fabiobaltieri@google.com
(cherry picked from commit 56dcafe)

C++ file compilation is actually missing the warning-as-error handling,
causing warnings in build files to be unnoticed in CI. Add a flag to
handle them as well.

Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
(cherry picked from commit 56dcafe)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants