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

Support Valgrind Helgrind XML Report #640

Closed
curtpm opened this issue Oct 12, 2015 · 8 comments
Closed

Support Valgrind Helgrind XML Report #640

curtpm opened this issue Oct 12, 2015 · 8 comments
Assignees
Labels
Milestone

Comments

@curtpm
Copy link

curtpm commented Oct 12, 2015

when running valgrind with the helgrind tool "valgrind --tool=helgrind" the final xml reports are not picked up.

Could not add the issue, details: 'The rule 'valgrind:Race' does not exist.'

Is there a way to add a CustomRule to get some kinda of basic reporting for these kind of errors?

@jmecosta
Copy link
Member

Each sensor supports custom rules via template rule. You just need to find
it in the profile and create a new one with name, you can try that

On Tue, Oct 13, 2015, 00:25 Curtis Mahieu [email protected] wrote:

when running valgrind with the helgrind tool "valgrind --tool=helgrind"
the final xml reports are not picked up.

Could not add the issue, details: 'The rule 'valgrind:Race' does not exist.'

Is there a way to add a CustomRule to get some kinda of basic reporting
for these kind of errors?


Reply to this email directly or view it on GitHub
#640.

@curtpm
Copy link
Author

curtpm commented Oct 13, 2015

Thanks for the reply, i did setup a custom rule from the valgrind template for a "Race" type.
It works fine for basic support.

Is there any way to get the output better formatted? like the normal valgrind report it seems to condense the entire stack trace onto a single line and makes it very hard to read.

@jmecosta
Copy link
Member

That I am ensure, can you try formating the messages before sonar? We also
have this problem in our Intel inspector report and I think the sonar ui
might just limited. I remember in old versions it was possible to provide
big messages for parse errors but its not the case anymore

On Tue, Oct 13, 2015, 19:28 Curtis Mahieu [email protected] wrote:

Thanks for the reply, i did setup a custom rule from the valgrind template
for a "Race" type.
It works fine for basic support.

Is there any way to get the output better formatted? like the normal
valgrind report it seems to condense the entire stack trace onto a single
line and makes it very hard to read.


Reply to this email directly or view it on GitHub
#640 (comment).

@guwirth guwirth added this to the M 0.9.4 milestone Oct 24, 2015
@guwirth guwirth closed this as completed Oct 24, 2015
@curtpm
Copy link
Author

curtpm commented Oct 26, 2015

I know this issue is closed, but sonar 5.2 has support for an issue to reference multiple locations in a code base.

This is exactly what helgrind data needs, the xml output is an array of locations where the data race was detected. Any plans on adding some kind of support for this?

I also wouldn't mind trying to make an attempt to implement it, but i might need some guidance.

@jmecosta
Copy link
Member

Can you point out where this feature is documented in 5.2.

@curtpm
Copy link
Author

curtpm commented Oct 27, 2015

https://groups.google.com/forum/#!msg/sonarqube/ZCIsvd41Esk/tHZOUEXfDwAJ

- More features are available to better manage issues
        Support for more precise issue location and multiple location (<= this requires language plugins to provide the information)

@jmecosta
Copy link
Member

i think this is what you are looking for: https://jira.sonarsource.com/browse/SONAR-4865
is the race issue occurring only on a single file, because if i understand correctly that's mandatory. if a stacktrace is to be saved then i dont think this will be supported

@curtpm
Copy link
Author

curtpm commented Oct 27, 2015

OK, thanks for the reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

3 participants