Re: [cdt-dev] CDT and GitHub (was: RE: Unit testing support for Eclipse

[Marc Khouzam <marc.khouzam@xxxxxxxxxxxx>]

> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx 
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Andrew Overholt
> Sent: Wednesday, August 31, 2011 3:55 PM
> To: Schaefer, Doug
> Cc: CDT General developers list.
> Subject: Re: [cdt-dev] CDT and GitHub (was: RE: Unit testing 
> support for Eclipse CDT)
> 
> * Schaefer, Doug <Doug.Schaefer@xxxxxxxxxxxxx> [2011-08-31 11:54]:
> > So, Andrew, are saying is that on a fetch (from the github repo in
> > this case to the local repo from which I'll push to 
> eclipse.org after
> > the merge), all the new commit records get updated so that the
> > Committer field is set to the name of the person doing the fetch?
> 
> Here's a concrete example from Linux Tools that may help.  The bug:
> 
>   Create unit tests for Helgrind
>   https://bugs.eclipse.org/bugs/show_bug.cgi?id=345863
> 
> The contributor's GitHub commit (referenced in the bug):
>   
> https://github.com/danielhb/linuxtools/commit/3103bea0dceb75a2
> 2e65f716d7cfc602a124586e

Just to be sure, when you fetch such a commit, you will actually
get a set of commits, if the contributor made his/her change
using multiple commits.  That way, we keep the history.  Right?

> I fetched from there and pushed into Linux Tools, resulting in:
>   
> http://git.eclipse.org/c/linuxtools/org.eclipse.linuxtools.git
> /commit/?id=cdd81a6ee600ef97e14fa191743415ea6457f576
> 
> As you can see, the committer is set to me and the author 
> remains as the contributor.

What is not clear is how the CQ is handled.  In your case,
you attached an actual patch to the CQ.  Normally, we won't
have such a patch.

http://wiki.eclipse.org/Development_Resources/Handling_Git_Contributions
mentions to include "the URL of the ref" of the commit.
But if the change is actually is a set of commit should we specify
the first and last commit maybe?  So that the IP reviewer knows what
is the new code?

Maybe I'm mis-understanding something....

> 
> HTH,
> 
> Andrew
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>