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

[feat] Accept non-existing file paths in report-converter #4288

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bruntib
Copy link
Contributor

@bruntib bruntib commented Jul 8, 2024

Some analyzer tools (e.g. sanitizers) give an output that contains relative source file paths. When report-converter is executed from a directory where these absolute paths are not valid, then reports are simply not generated. If all reports contain such a relative path, then this message is given by report-converter:

No 'lsan' results can be found in '...'

This message is obviously false.

This patch allows non-existing file paths in the analyzer outputs. Using further CodeChecker commands (store, parse, etc.) will emit an error message on these paths anyway, but at least we get rid of the incomprehensible error message above.

Fixes #4280

Some analyzer tools (e.g. sanitizers) give an output that contains
relative source file paths. When report-converter is executed from a
directory where these absolute paths are not valid, then reports are
simply not generated. If all reports contain such a relative path, then
this message is given by report-converter:

No 'lsan' results can be found in '...'

This message is obviously false.

This patch allows non-existing file paths in the analyzer outputs. Using
further CodeChecker commands (store, parse, etc.) will emit an error
message on these paths anyway, but at least we get rid of the
incomprehensible error message above.

Fixes Ericsson#4280
@bruntib bruntib added CLI 💻 Related to the command-line interface, such as the cmd, store, etc. commands report-converter 🔄 labels Jul 8, 2024
@bruntib bruntib added this to the release 6.25.0 milestone Jul 8, 2024
@bruntib bruntib requested a review from vodorok as a code owner July 8, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLI 💻 Related to the command-line interface, such as the cmd, store, etc. commands report-converter 🔄
Projects
None yet
Development

Successfully merging this pull request may close these issues.

report-converter [INFO] No 'lsan' results can be found
1 participant