Skip to content

refs #13532 - test/cli/clang-import_test.py: pass -qto some invocations#7643

Merged
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:cl-imp-q
Jul 6, 2025
Merged

refs #13532 - test/cli/clang-import_test.py: pass -qto some invocations#7643
firewave merged 1 commit intocppcheck-opensource:mainfrom
firewave:cl-imp-q

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Jul 4, 2025

No description provided.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Jul 4, 2025

I had a hard time to get #6855 to pass and it mostly failed with https://trac.cppcheck.net/ticket/13532#comment:6. It failed because output is generated by the parent process and spawned one which might lead to differences in the order it is received by the output. The regular output is not of interest in the tests so we can just disable it and thus get rid of the potential race.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Jul 4, 2025

This happened on macos-15 which has less processors available as the other runners so that will obvious change the behavior of the tests which are run in parallel.

@firewave firewave merged commit 0eb4dd8 into cppcheck-opensource:main Jul 6, 2025
60 checks passed
@firewave firewave deleted the cl-imp-q branch July 6, 2025 19:35
@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Jul 6, 2025

I merged this because it fixes flakey macos-15 builds in the CI.

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Jul 6, 2025

Ugh - I missed the typo in the title ...

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.

1 participant