Re: [cdt-dev] Rx/Tx tracing patch

[Jason Litton <jason@xxxxxxxxxxxxxxx>]

Well, it builds and launches, but it doesn't work correctly. I didn't see the trace option available in the menu. I'll test this all through Juno. I'm pretty sure I've got the latest release around here somewhere...

----- Original Message -----
From: "Marc Khouzam" <marc.khouzam@xxxxxxxxxxxx>
To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
Sent: Friday, May 18, 2012 12:34:16 PM
Subject: Re: [cdt-dev] Rx/Tx tracing patch

I'm not sure when the extension point was added, but 
I wouldn't be surprised if it wasn't in Juno.

Either way, to provide a CDT contribution that you
have properly tested, you really should be on the
latest platform.
In fact, I don't expect CDT to be compiling with
Indigo anymore, or does it? 

> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx 
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Jason Litton
> Sent: Friday, May 18, 2012 2:27 PM
> To: CDT General developers list.
> Subject: Re: [cdt-dev] Rx/Tx tracing patch
> 
> I think you're right. I need the xml changes. However, I'm 
> trying to write them in and I'm getting an unknown extension 
> point on org.eclipse.ui.trace.traceComponents. Is this 
> because I'm still using Indigo as my dev environment, or do I 
> need to pull in that in some other way to get the patch to 
> work? I'm working through egit woes, and I understand how to 
> push to the same gerrit branch, so that should clear that part up.
> 
> ----- Original Message -----
> From: "Marc Khouzam" <marc.khouzam@xxxxxxxxxxxx>
> To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
> Sent: Friday, May 18, 2012 11:05:27 AM
> Subject: Re: [cdt-dev] Rx/Tx tracing patch
> 
> > -----Original Message-----
> > From: cdt-dev-bounces@xxxxxxxxxxx 
> > [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Jason Litton
> > Sent: Friday, May 18, 2012 12:51 PM
> > To: CDT General developers list.
> > Subject: [cdt-dev] Rx/Tx tracing patch
> > 
> > I'm trying to work on fixing the patch for the RxTx tracing. 
> > I've added copyright to the files. Would you like me to add 
> > the extension points to plugin.xml and the plugin properties 
> > stuff, or should that be a separate patch? 
> 
> I assumed you needed the plugin.xml parts to achieve your goal
> of saving the traces to a file.  If that is true, might as
> well put everything as a single patch.
> 
> If you don't absolutely need the plugin.xml stuff to achieve 
> your goal then two separate patches is ok (but not necessary).
> That means two Gerrit reviews and two bugzillas.
> 
> > Also, this is my 
> > first shot at using Git and Gerrit, and it's not going well, 
> > so I may have an ugly series of commits.
> 
> There should be only one commit for a Gerrit review.  That
> commit keeps getting updated with new changes.
> To update an existing review, as you need to do, you should
> modify your previous commit.  To do that with Egit, you
> simply start to commit your new changes, but in the
> commit message dialog, click on the "Ammend Previous Commit"
> toggle button, which is on the top-right.
> This will automatically use the Gerrit changeId that was
> used in the previous Gerrit review, which is how Gerrit
> knows that this updated commit should go to the same review.
> 
> Feel free to ask more questions if you need to.
> 
> Marc
> 
> 
> > Thanks,
> > Jason
> > _______________________________________________
> > cdt-dev mailing list
> > cdt-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/cdt-dev
> > 
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> 
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev