-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Fix #13841 (Update simplecpp to 1.4.1) #7531
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
Conversation
@danmar After this change, my cppcheck CI job started failing with |
@SChernykh ouch! I fail to reproduce directly. Would it be possible to create a small minimal example? Here is a quick test in cppcheck repo:
the source file includes various files in externals and lib. |
This is the last passed CI check in my repo: https://github.com/SChernykh/p2pool/actions/runs/15063299959 Passed CI check used commit 6d7dc76 So somewhere between these two commits things got broken. I suspect it was this one, because other commits only touch tests. Edit: and I didn't do any commits between these two CI runs - the second run was a nightly scheduled check. |
@Tal500 would it be possible for you to take a look if this is related to your changes? |
Yes, I'll try to look on thus tomorrow |
Fixed in #429 |
When is simplecpp going to be updated here? It's still giving me the same error because code is not updated. |
the fix has not been merged in simplecpp yet: |
No description provided.