Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 344504

Summary: Contributing a Helgrind plugin for Linux Tools Project
Product: [Tools] Linux Tools Reporter: Daniel Henrique Barboza <danielhb>
Component: ValgrindAssignee: Elliott Baron <ebaron>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: overholt, wainersm
Version: unspecified   
Target Milestone: 0.8.0   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on:    
Bug Blocks: 346455, 348269, 354223    
Attachments:
Description Flags
Tarball with the source code of Helgrind plugin overholt: iplog+

Description Daniel Henrique Barboza CLA 2011-05-02 14:44:58 EDT
Build Identifier: 

I've developed a initial version of a Helgrind plugin for Eclipse.

Basically it works pretty much like the existing Valgrind plugins. I've tested it under Valgrind 3.6.0 and Eclipse 3.6.2. There are more work to be done, but I think it's good to go for a initial go.

I'm attaching the tarball of the source code in this bug.

Reproducible: Always
Comment 1 Daniel Henrique Barboza CLA 2011-05-02 14:46:03 EDT
Created attachment 194525 [details]
Tarball with the source code of Helgrind plugin

Tarball with the source code of Helgrind plugin
Comment 2 Andrew Overholt CLA 2011-05-02 14:58:37 EDT
Thanks for the contribution, Daniel!  I've opened a contribution questionnaire [1] with the Eclipse legal team:

http://dev.eclipse.org/ipzilla/show_bug.cgi?id=5116

(Only existing committers will be able to see the contents of that URL)

I'm confident that they're going to ask that you state that you authored 100% of the code and if not we'll need contact details for each of the contributors: name, organization, email address and/or phone number, percent of contribution authored.

IBM being a member organization makes this easier.

[1]
http://wiki.eclipse.org/Development_Resources/New_Commmitter_Handbook#Contribution_Questionnaire_.28CQ.29
Comment 3 Elliott Baron CLA 2011-05-02 15:50:21 EDT
Fantastic! I'll be glad to help in any way I can.
Comment 4 Daniel Henrique Barboza CLA 2011-05-03 13:12:03 EDT
Just to double-check: I need to wait for the contact of the Eclipse Legal team and answer their questions, right? Are they going to contact by email?
Comment 5 Andrew Overholt CLA 2011-05-03 13:16:07 EDT
(In reply to comment #4)
> Just to double-check: I need to wait for the contact of the Eclipse Legal team
> and answer their questions, right? Are they going to contact by email?

That is correct.  Thanks, Daniel.
Comment 6 Andrew Overholt CLA 2011-05-11 17:22:40 EDT
Hi Daniel,

Sharon from the Eclipse legal team has requested a statement from you (on this bug) confirming that you:

1.  authored 100% of the content I am contributing 
2.  have the rights to contribute the content to Eclipse 
3.  contribute the content under the project license (EPL in this case)

Also, if anyone other than yourself wrote any of the code, they'll need to put a similar statement here.

Thanks.
Comment 7 Daniel Henrique Barboza CLA 2011-05-12 10:41:16 EDT
I, Daniel Henrique Barboza, assert that I am the author of 100% of the code I am contributing, I have the rights to contribute this content to Eclipse and that such content is being contributed under the terms of EPL.
Comment 8 Andrew Overholt CLA 2011-05-13 09:10:00 EDT
Thanks, Daniel.  The awesome Sharon of the Eclipse Legal team has managed to squeeze this in and has said "This submission is approved!"  We can now commit this to the repository.

Elliott, did you want to commit it or shall I?
Comment 9 Elliott Baron CLA 2011-05-13 14:17:41 EDT
(In reply to comment #8)
> Thanks, Daniel.  The awesome Sharon of the Eclipse Legal team has managed to
> squeeze this in and has said "This submission is approved!"  We can now commit
> this to the repository.
> 
> Elliott, did you want to commit it or shall I?

Great! I'll commit it.
Comment 10 Elliott Baron CLA 2011-05-13 15:07:24 EDT
(In reply to comment #9)
> (In reply to comment #8)
> > Thanks, Daniel.  The awesome Sharon of the Eclipse Legal team has managed to
> > squeeze this in and has said "This submission is approved!"  We can now commit
> > this to the repository.
> > 
> > Elliott, did you want to commit it or shall I?
> 
> Great! I'll commit it.

I have checked the code in and added the Helgrind plugin to the Valgrind feature.
Thanks Daniel!

Andrew, this plugin needs a pom file for Tycho, correct? Is it just a matter of copying from another plugin and doing some renaming?
Comment 11 Andrew Overholt CLA 2011-05-13 16:07:34 EDT
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > Thanks, Daniel.  The awesome Sharon of the Eclipse Legal team has managed to
> > > squeeze this in and has said "This submission is approved!"  We can now commit
> > > this to the repository.
> > > 
> > > Elliott, did you want to commit it or shall I?
> > 
> > Great! I'll commit it.
> 
> I have checked the code in and added the Helgrind plugin to the Valgrind
> feature.
> Thanks Daniel!
> 
> Andrew, this plugin needs a pom file for Tycho, correct? Is it just a matter of
> copying from another plugin and doing some renaming?

Copying from another plugin and doing renaming is easiest.  You can then add it to valgrind/pom.xml.

Thanks for taking care of this checkin, Elliott.
Comment 12 Elliott Baron CLA 2011-05-13 16:36:45 EDT
I created the POM file and added the module entry to the parent POM file. I'll mark this resolved, feel free to reopen if we're not done.
Comment 13 Andrew Overholt CLA 2011-05-15 20:26:48 EDT
Thanks, Elliott.  I see the build from yesterday includes the plugin jar so it looks like everything went well.

I created bug #345863 to track creating some unit tests.
Comment 14 Wainer dos Santos Moschetta CLA 2011-05-16 09:14:20 EDT
Hi Andrew,

Just to double check: helgrind plug-in will not be include in Linux Tools 0.8?
Comment 15 Andrew Overholt CLA 2011-05-16 10:41:47 EDT
(In reply to comment #14)
> Hi Andrew,
> 
> Just to double check: helgrind plug-in will not be include in Linux Tools 0.8?

Actually, now that it's been approved by the legal team, we can get it in.  Is there a reason it should not be included?
Comment 16 Wainer dos Santos Moschetta CLA 2011-05-16 10:47:51 EDT
(In reply to comment #15)
> (In reply to comment #14)
> > Hi Andrew,
> > 
> > Just to double check: helgrind plug-in will not be include in Linux Tools 0.8?
> 
> Actually, now that it's been approved by the legal team, we can get it in.  Is
> there a reason it should not be included?

No, absolutely not. Actually I'd like to consume it through Linux Tools 0.8. Please, go ahead to get it in.

Thanks for all your support.
Comment 17 Andrew Overholt CLA 2011-05-16 11:58:20 EDT
Daniel,

Could you please write up a few sentences and take a screenshot or two for use in our new and noteworthy for 0.8?  You can see examples of other such entries here:

  http://www.eclipse.org/linuxtools/new/

Thanks,

Andrew
Comment 18 Daniel Henrique Barboza CLA 2011-05-16 12:05:29 EDT
Is Helgrind plugin going live on 0.8 already? This is AWESOME! Great news indeed!

@ Andrew

I'm on it
Comment 19 Andrew Overholt CLA 2011-05-19 10:22:57 EDT
*** Bug 273876 has been marked as a duplicate of this bug. ***