Thread (69 messages) 69 messages, 17 authors, 2011-02-07

[PATCH 0/7] Nexus One Support

From: joe@perches.com (Joe Perches)
Date: 2011-01-21 02:25:34
Also in: linux-arm-msm, lkml

On Thu, 2011-01-20 at 17:58 -0800, Daniel Walker wrote:
On Thu, 2011-01-20 at 17:41 -0800, Joe Perches wrote:
quoted
On Thu, 2011-01-20 at 16:55 -0800, Daniel Walker wrote:
quoted
On Thu, 2011-01-20 at 16:42 -0800, Dima Zavin wrote:
quoted
You are not the author of any of these patches. Where are the author
attributions for the team that actually wrote this code?
In the commit text.. The author field is used to denote who authored the
commit, which in this case is me.
You have that wrong.
Author and Committer are different git fields.
http://www.kernel.org/pub/software/scm/git/docs/user-manual.html

      * an author: The name of the person responsible for this change,
        together with its date.
      * a committer: The name of the person who actually created the
        commit, with the date it was done. This may be different from
        the author, for example, if the author was someone who wrote a
        patch and emailed it to the person who used it to create the
        commit.
I'm not even sure how to make these different, but in this case it
doesn't matter because the "committer" as you defined it above is more
than one person ..
Not really, no.

The authors may be different, but the first git
committer of the patch is different.

The committer is the person that does a git commit
either directly with git commit or git am.

If a git tree is pulled by someone else, the initial
committer name remains on the commit.

You should keep the original patch author names and
add your own "Signed-off-by:" and not claim authorship
of the patches themselves.

cheers, Joe
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help