Skip to content

Conversation

gruenich
Copy link
Contributor

No description provided.

@firewave
Copy link
Collaborator

We need to bail out in the CI on unused CMake options (by passing -Werror=dev?). I am not sure if I have not filed a ticket about this yet.

@firewave
Copy link
Collaborator

We need to bail out in the CI on unused CMake options (by passing -Werror=dev?). I am not sure if I have not filed a ticket about this yet.

I filed https://trac.cppcheck.net/ticket/14081 about that. But that cannot be easily done right now because of https://trac.cppcheck.net/ticket/14082.

@firewave
Copy link
Collaborator

firewave commented Aug 20, 2025

Unfortunately it appears you cannot fail the build on unused manually-specified variables.

@gruenich
Copy link
Contributor Author

What is the issue with 14082? It behaves as expected.

@firewave
Copy link
Collaborator

What is the issue with 14082? It behaves as expected.

It does not matter because it is outside of the scope of this.

Copy link

@firewave firewave merged commit 98a168b into danmar:main Aug 26, 2025
63 checks passed
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.

2 participants