From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Ericsson Subject: Re: git push mirror mode Date: Thu, 08 Nov 2007 14:14:12 +0100 Message-ID: <47330BA4.6030101@op5.se> References: <20071108121136.GG9736@shadowen.org> <20071108124435.GH9736@shadowen.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: Johannes Schindelin , git@vger.kernel.org, Junio C Hamano To: Andy Whitcroft X-From: git-owner@vger.kernel.org Thu Nov 08 14:14:42 2007 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1Iq7DS-0007sx-GR for gcvg-git-2@gmane.org; Thu, 08 Nov 2007 14:14:38 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754625AbXKHNOR (ORCPT ); Thu, 8 Nov 2007 08:14:17 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754633AbXKHNOR (ORCPT ); Thu, 8 Nov 2007 08:14:17 -0500 Received: from mail.op5.se ([193.201.96.20]:43611 "EHLO mail.op5.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754456AbXKHNOQ (ORCPT ); Thu, 8 Nov 2007 08:14:16 -0500 Received: from localhost (localhost.localdomain [127.0.0.1]) by mail.op5.se (Postfix) with ESMTP id 28B4B1F0870A; Thu, 8 Nov 2007 14:14:17 +0100 (CET) X-Virus-Scanned: amavisd-new at X-Spam-Flag: NO X-Spam-Score: -2.499 X-Spam-Level: X-Spam-Status: No, score=-2.499 tagged_above=-10 required=6.6 tests=[BAYES_00=-2.599, RDNS_NONE=0.1] Received: from mail.op5.se ([127.0.0.1]) by localhost (mail.op5.se [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id smLT7a+qv1IZ; Thu, 8 Nov 2007 14:14:16 +0100 (CET) Received: from nox.op5.se (unknown [192.168.1.20]) by mail.op5.se (Postfix) with ESMTP id E9B871F08708; Thu, 8 Nov 2007 14:14:15 +0100 (CET) User-Agent: Thunderbird 2.0.0.5 (X11/20070727) In-Reply-To: <20071108124435.GH9736@shadowen.org> Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: Andy Whitcroft wrote: > On Thu, Nov 08, 2007 at 12:19:18PM +0000, Johannes Schindelin wrote: >> Hi, >> >> On Thu, 8 Nov 2007, Andy Whitcroft wrote: >> >>> Ok, sometime back Junio sent out a proof-of-concept change to >>> send-pack allowing a mirror mode. >> You added/left his sign-off, but did not attribute the patches to him. >> Why? > > I believe I left his signed off by from the original (first) patch, and > added mine to indicate that what I had modified was also unecombered. > The second patch is only signed off by me as I am the author. In my > world (admittedly a kernel hacker) the first Signed-off-by: indicates the > primary authorship of that patch and the [apw@...] part tries to clarify > the changes I made therein. > > No intentional stripping of credit was intended, and I believe that the > attribution as written states Junio is the originator of this patch. > However that is the way I would read the meanings of these lines, if git > has different rules or you think there is a clearer way of stating this > I am happy to change it, and resend it so attributed. > Barring any errors in my understanding of the matter, here's how it works for git. git separates author from committer, so code attribution is done with author, and "I verified this is sane" is done by committer. Those two usually only ever differ when the user tells git commit that the author was someone else than him/her self, or when rewriting history with git rebase or similar. git am also maintains authorship (using the From: line in emails), but sets $committer to the person running it, so when you apply patches sent by email from someone else you get the code attribution right by default. The Signed-off-by line is, in git, used as "I touched the code here and agree that it may be included in the mothership repo and all future releases" (the spirit of that sentence is also in Documentation/SubmittingPatches). We also have Acked-by (as does the kernel, no? I think we inherited it from there) to mean something along the lines of "I vote we include this", but not always based on technical merit (ie, patches can have many acks without having ever been tested). Suggested-by, Tested-by and Reported-by are used less often, not always written in dash-form, but hopefully always self-explanatory ;-) -- Andreas Ericsson andreas.ericsson@op5.se OP5 AB www.op5.se Tel: +46 8-230225 Fax: +46 8-230231