Skip to content

Conversation

@barnabasdomozi
Copy link
Collaborator

Currently, it is a problem to determine if the analysis was successful or failed during execution.

With this patch, we will create plist.err files to indicate a failed analysis action.
The parse --status command was also introduced to
get the analysis summary of an existing report directory.

@barnabasdomozi barnabasdomozi added the analyzer 📈 Related to the analyze commands (analysis driver) label Oct 2, 2025
@barnabasdomozi barnabasdomozi added this to the release 6.27.0 milestone Oct 2, 2025
@bruntib
Copy link
Contributor

bruntib commented Oct 3, 2025

Aren't .err files generated only when analysis fails?
Why clangsa and clang-tidy are hard-coded when printing stats?

@barnabasdomozi barnabasdomozi force-pushed the plist_error branch 2 times, most recently from ba24441 to 2bf9f70 Compare October 3, 2025 14:11
@barnabasdomozi
Copy link
Collaborator Author

@bruntib

Aren't .err files generated only when analysis fails?

If the return_code is 0 we return from error_file.py's create() method and no .plist.err will be generated.

Why clangsa and clang-tidy are hard-coded when printing stats?

Is there a list of supported analyzers defined somewhere else?
I think in the regex pattern we should try to match for existing analyzers and skip the rest of the files.

@barnabasdomozi barnabasdomozi force-pushed the plist_error branch 4 times, most recently from f8bebc6 to b0d8ea1 Compare October 7, 2025 15:06
Currently, it is a problem to determine if the analysis
was successful or failed during execution.

With this patch, we will create plist.err files to
indicate a failed analysis action.
The parse --status command was also introduced to
get the analysis summary of an existing report directory.
@barnabasdomozi
Copy link
Collaborator Author

@bruntib

I added error file generation for the other analyzers as well.

@barnabasdomozi barnabasdomozi marked this pull request as draft November 3, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

analyzer 📈 Related to the analyze commands (analysis driver)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants