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

[analyze] Fix duplication warning when creating failed zip #3213

Merged

Conversation

csordasmarton
Copy link
Contributor

Closes #3001

When there is a compilation failure, we will try to create a failed zip file
with source files. It is possible that we will try to add the same file
multiple times but on different path: a/b/c and /a/b/../b/c. In this
case CodeChecker will print out the following warning message:
UserWarning: Duplicate name:...
To avoid this problem we normalize the path before inserting it to the
zip file.

When there is a compilation failure, we will try to create a failed zip file
with source files. It is possible that we will try to add the same file
multiple times but on different path: `a/b/c` and `/a/b/../b/c`. In this
case CodeChecker will print out the following warning message:
`UserWarning: Duplicate name:...`
To avoid this problem we normalize the path before inserting it to the
zip file.
@csordasmarton csordasmarton added bugfix 🔨 analyzer 📈 Related to the analyze commands (analysis driver) labels Feb 24, 2021
@bruntib bruntib merged commit fef103a into Ericsson:master Feb 24, 2021
@csordasmarton csordasmarton deleted the fix_duplicate_warning_in_failed_zip branch March 10, 2021 09:59
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 🔨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

analyze --capture-analysis-output writes corrupt zipfiles
3 participants