From patchwork Thu Aug 9 20:51:33 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Luis R. Rodriguez" X-Patchwork-Id: 176272 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 1CA222C007F for ; Fri, 10 Aug 2012 06:51:56 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759111Ab2HIUvm (ORCPT ); Thu, 9 Aug 2012 16:51:42 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:40527 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751268Ab2HIUvj (ORCPT ); Thu, 9 Aug 2012 16:51:39 -0400 Received: by pbbrr13 with SMTP id rr13so1468759pbb.19 for ; Thu, 09 Aug 2012 13:51:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:x-mailer; bh=j0tcHu4VgDvNHGAE956PN2oQBvvyBPNl17j4FM/ShhU=; b=q2salR/MRkDozuCtkyTnYdNM2FgcwlbtDPqPsjN5A+pn/gkruaTiDDLAcj6Y3/Hft0 skxV0mYyY8wWLZonnBUmXrT4cJrlN8NFra+RSNhqAuAQhJmXj1geMh3V44v4q2LGm3Wn SwYFgkovgQKJuN/Cch4YB7HCAsZOM7fTaIBg+n538d5gsB/nXGN0v6kpsAddAJOC5VCw xYygCEE3YtmC54f9TscYrmdw3vahcpA1BZoVa6gviQEf63Bd+X0kyg0/F4ykEEpWcz4B g7xHjro9ged900qVv2jaEzgqjc74TRKrFTmNO+MwY3HUn3K+//z9joYHw0mdILWZblqs 4vhg== Received: by 10.68.212.161 with SMTP id nl1mr6981288pbc.84.1344545498924; Thu, 09 Aug 2012 13:51:38 -0700 (PDT) Received: from mcgrof@gmail.com (c-24-7-61-223.hsd1.ca.comcast.net. [24.7.61.223]) by mx.google.com with ESMTPS id wn1sm1789161pbc.57.2012.08.09.13.51.36 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 09 Aug 2012 13:51:38 -0700 (PDT) Received: by mcgrof@gmail.com (sSMTP sendmail emulation); Thu, 09 Aug 2012 13:51:35 -0700 From: "Luis R. Rodriguez" To: torvalds@linux-foundation.org Cc: rdunlap@xenotime.net, tytso@mit.edu, alan@lxorguk.ukuu.org.uk, davem@davemloft.net, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "Luis R. Rodriguez" Subject: [PATCH] SubmittingPatches: clarify SOB tag usage when evolving submissions Date: Thu, 9 Aug 2012 13:51:33 -0700 Message-Id: <1344545493-6820-1-git-send-email-mcgrof@do-not-panic.com> X-Mailer: git-send-email 1.7.10.4 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: "Luis R. Rodriguez" Initial large code submissions typically are not accepted on their first patch submission. The developers are typically given feedback and at times some developers may even submit changes to the original authors for integration into their second submission attempt. Developers wishing to contribute changes to the evolution of a second patch submission must supply their own Siged-off-by tag to the original authors and must submit their changes on a public mailing list or ensure that these submission are recorded somewhere publicly. To date a few of these type of contributors have expressed different preferences for whether or not their own SOB tag should be used for a second code submission. Lets keep things simple and only require the contributor's SOB tag if so desired explicitly. It is not technically required if there already is a public record of their contribution somewhere. Document this on Documentation/SubmittingPatches Signed-off-by: Luis R. Rodriguez Signed-off-by: The patch went through you. Legal responsibility. Acked-by: You know what you are talking about and approve. Reviewed-by: You reviewed the patch and approve. --- Documentation/SubmittingPatches | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches index c379a2a..e018043 100644 --- a/Documentation/SubmittingPatches +++ b/Documentation/SubmittingPatches @@ -366,6 +366,21 @@ and protect the submitter from complaints. Note that under no circumstances can you change the author's identity (the From header), as it is the one which appears in the changelog. +If you are submitting a large change (for example a new driver) at times +you may be asked to make quite a lot of modifications prior to getting +your change accepted. At times you may even receive patches from developers +who not only wish to tell you what you should change to get your changes +upstream but actually send you patches. If those patches were made publicly +and they do contain a Singed-off-by tag you are not expected to provide +their own Singed-off-by tag on the second iteration of the patch so long +as there is a public record somewhere that can be used to show the +contributor had sent their changes with their own Singed-off-by tag. + +If you receive patches privately during development you may want to +ask for these patches to be re-posted publicly or you can also decide +to merge the patches as part of a separate historical git tree that +will remain online for historical archiving. + Special note to back-porters: It seems to be a common and useful practise to insert an indication of the origin of a patch at the top of the commit message (just after the subject line) to facilitate tracking. For instance,