Skip to content

Conversation

@firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

Requires #8198 and #8159 to be merged first.

@firewave firewave changed the title CI-unixish.yml: use builddir consistently and verify usage of cached results in selfcheck use builddir consistently and verify usage of cached results in selfcheck Feb 12, 2026
selfcheck.sh: make it possible to pass additional flags

selfcheck.sh: removed unnecessary early exit

selfcheck.sh: remove generated folders

extracted selfcheck into script
@firewave firewave force-pushed the selfcheck-builddir-x branch from 2216ede to 5e6e050 Compare February 12, 2026 16:47
@firewave firewave changed the title use builddir consistently and verify usage of cached results in selfcheck use builddir consistently and verify usage of cached results in selfcheck / improved --debug-analyzerinfo output Feb 12, 2026
@firewave firewave force-pushed the selfcheck-builddir-x branch from 5e6e050 to ad64df7 Compare February 12, 2026 16:56
@firewave
Copy link
Collaborator Author

discarding cached result from 'b1/main.a1' for 'cli/main.cpp' - hash mismatch (16146913026568202460 / 18076949867239869788)
discarding cached result from 'b1/main.a1' for 'gui/main.cpp' - hash mismatch (18076949867239869788 / 16146913026568202460)

The problem is that we perform individual analysis(eses) on the same builddir leading to conflicts when a file with the same name exists in a different location.

I am not sure this can be resolved as the files.txt is written unconditionally essentially discarding.

A valid use case would be the usage of a cache in conjunction with an IDE integration which would only invoke the analysis for a single file. In that case though we could generate the files.txt for the whole project/inputs regardless of the selected subset. But that would only work with a fixed project configuration but not in the case of this PR when you extract configurations.

It could be achieved though by not writing the file unconditionally but use it as an index for lookups and append new entries. This would leave outdated and/or removed files behind but that is already an issue now (needs a separate ticket).

@firewave firewave force-pushed the selfcheck-builddir-x branch from ad64df7 to ade4d58 Compare February 12, 2026 18:57
@sonarqubecloud
Copy link

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