Skip to content
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

[analyzer][clang][debug] Fix prepare_analyzer_cmd default configuration #4109

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

gamesh411
Copy link
Collaborator

@gamesh411 gamesh411 commented Dec 4, 2023

The default value of ctu-dir and the printing of prepared analyzer
command insteam of writing it to a file was causing issues.
This fixes the script for trivial open source projects.

@gamesh411 gamesh411 added bugfix 🔨 dev env ⛑️ Development environment analyzer 📈 Related to the analyze commands (analysis driver) clang sa 🐉 The Clang Static Analyzer is a source code analysis tool that finds bugs in C-family programs. WIP 💣 Work In Progress labels Dec 4, 2023
The default value of ctu-dir and the printing of prepared analyzer
command insteam of writing it to a file was causing issues.
This fixes the script for trivial open source projects.
@gamesh411 gamesh411 removed the WIP 💣 Work In Progress label Dec 5, 2023
@gamesh411
Copy link
Collaborator Author

Tested with tinyxml2, and these configuration defaults and the file creation behaviour is now in-line with the documentation.

@gamesh411 gamesh411 changed the title [analyzer][clang][debug] Fix analyzer crash debug scripts [analyzer][clang][debug] Fix prepare_analyzer_cmd default configuration Dec 5, 2023
@bruntib bruntib merged commit b8f396f into Ericsson:master Dec 5, 2023
7 of 8 checks passed
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) bugfix 🔨 clang sa 🐉 The Clang Static Analyzer is a source code analysis tool that finds bugs in C-family programs. dev env ⛑️ Development environment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants