Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3;
Message-ID: <5258664f-1f1f-6f3d-33ca-c3b2d60601b6@intel.com>
Date:   Mon, 23 Oct 2023 11:24:49 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.3.1
Subject: Re: [PATCH] checkpatch: allow tags between co-developed-by and their
 sign-off
To:     Joe Perches <joe@perches.com>, Andy Whitcroft <apw@canonical.com>,
        Dwaipayan Ray <dwaipayanray1@gmail.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>
CC:     Jacob Keller <jacob.e.keller@intel.com>,
        <linux-kernel@vger.kernel.org>
References: <20231020132156.37882-1-przemyslaw.kitszel@intel.com>
 <fbd86466d325055846e5d63f62ca1dc8613ba80d.camel@perches.com>
Content-Language: en-US
From:   Przemek Kitszel <przemyslaw.kitszel@intel.com>
In-Reply-To: <fbd86466d325055846e5d63f62ca1dc8613ba80d.camel@perches.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WlduTHVrcy9tNEJua3pENzhWaWt5a21FdVpodjR3OXVRNUtZTXR3d0tIK0Zi?=
 =?utf-8?B?MldXK2tmRFFiSERnKyt4b09ib3dFY3dvUHhKTGxvdlBaWDNPelJlSGQ0aHhp?=
 =?utf-8?B?TUVnL2RwK096VDVGU0ZvMWozT096bnlta1ZKUVp0eE14ZElTWHZmeG5NNEd0?=
 =?utf-8?B?VWlBdUh1SmhXRC9DNzd2Q1pVd294R0drMzdJL0VlZjVGVkxPaW1Yb05CWEtF?=
 =?utf-8?B?YW9SbVVwZjJ4Ymg3VW0vUEVnRjdnL3pZOTdoUUN6Nm95SGdlN1FBMDkzT24w?=
 =?utf-8?B?M2RUODJNbEsyckRBOTJSTUdreEZ4UHF3eEhvVC9BL0tBZjIrRG4rZm9QRW90?=
 =?utf-8?B?Ykw0QnM4KyszY2JLZWpia1U2MFZaNHV5TWVjQVB1K0s2SzczQktJYklIVFcx?=
 =?utf-8?B?VEt6WkJDakMxdVpTUUZFUmJuQThaWE12UXcxSUdmdVAzWS8rSXVjK2NZaVRK?=
 =?utf-8?B?WDlZUEF1cXp6UVpKOVg5Rm5ZdDVmaWNVUkFQMGUwY2pOdzNEYTdpNEFXRTR6?=
 =?utf-8?B?UHVpVnNyVUVqMkZKb2ppQkNqeXZYWUViZEtmZTlXM3FDVVpYRkVCeWp4OVd0?=
 =?utf-8?B?T2ZMNE9OTjQ2ZzdJZmZvWTE5TVN4bGNBZzQvTnd6a2F5QlRuOGRLWU1WSU1z?=
 =?utf-8?B?UDBkbjJaZW54U3RJZzJUUzQxZ0FYY2dubUZaV051VlF4bUhGRTh0djZrM3pU?=
 =?utf-8?B?eTZvYUcyU3dleUpWMVhlT0ptQzlkb29CRTZoZkxvZ3AwcktCZ1ZnN0ZDaVZk?=
 =?utf-8?B?bzl4MWoxd3VhaHBJWGhjQzlTZzJXd1hiZGhYZWJ4enRodmhQTVJDMjFMdFhQ?=
 =?utf-8?B?aUt6K0I1WW15SysrMTU2REdCcVFCektRSEU1am9BSnROaVRlbnZXVzB5a1cz?=
 =?utf-8?B?TlE3bCsyUnFVN0kwL0hkdlVJNzQwNkRycTJXanpzK0pVcE5FVHJIaE9QamJH?=
 =?utf-8?B?Q2xnUUlGRGU0cjRkcGFYaDAzMjR2eHp0R3dnbHhkT2RTMXRvVGFkeXpBSVdi?=
 =?utf-8?B?YTdEa0hURFRFdU56K3dCMUVZTkk4S1NUL2N4eXIwaWtXSDNNaG5DNXVWN1Uw?=
 =?utf-8?B?enFMU3JPcGNKeWRMVW1MdTYzZjE3SjNkUTl3QkZHZmVEYVhZc2pDd2g1N01s?=
 =?utf-8?B?YVBlZjRmQWxQeGdlR29jYW5uMnNQNE9adm1ZTkpua0l3VHh1a0UvWDZ0Qkdq?=
 =?utf-8?B?cVFxemc4OWRkVTNxNmhPY0JrOW84UTQ0K1NxZUZndXlLRW95MWJObXdvQWhB?=
 =?utf-8?B?VS9VSnBPMmZaNVJLNzVNQ216amhnZmNpVndWK2VwTjBTVGVZOFBoeXNMa003?=
 =?utf-8?B?ajJlWS9pc3p2SHJ4a2s1d1ZWTWppcmpyOFBXRW0rb3pUVVJndE0xcHVndjNG?=
 =?utf-8?B?MnM1YjltTU1Gb01SSi9QYXdkdFFlUndzMzN4V0I1MTFZQnJScWFPV3NsVjd0?=
 =?utf-8?B?ZVBnKzBzZklmSlNlYlkyaU5pMFl1YXBnM1QzeElYVE9wSjF5R2xIbjZtbkVI?=
 =?utf-8?B?Q0c2cmNKbXVDT1BHdDEyMWtxcmVNOXRINlFQdnZiOHhVWk1UNVRmYTlBc1p5?=
 =?utf-8?B?UkhGbnhDSVN5elRSWlNyOVNRYXJBSGlOdVYrNjRVbVp5MGoya2xTVUpKL0Ry?=
 =?utf-8?B?elJSeFZFYm1TNmYyVTRtNFpaZXBhYmRFTHByL0RNMVdseTlCRlplUC9TcUYw?=
 =?utf-8?B?aHQ0ZlYvWmR2b0EvRm9ZTW04dnphVGt0RHBzdmdhWmlOTGR4ZVpEZVRIdmRj?=
 =?utf-8?B?VTF4d3htek9KKzlwSFNTTEROZHBmTkkzUW1saEpKM2FwV0piRnhJeFFjcUIr?=
 =?utf-8?B?UFZiUTdwaVJLb3A1azRJM1liSFZJQUFXa0VLdVFmaW40ZUR0anNmQ1k0TTQz?=
 =?utf-8?B?ZmtSSEp0SDh5N0dKK0tCNkllN21yckk4NXV5bmhkL0dOTTRZZ0R5OVFLaHVx?=
 =?utf-8?B?Tk8xRUt5Wndwd3hCQUlYUEFrYk5OVGNJYmZaVUxyVngrT3JnTkIxbUd1VVBY?=
 =?utf-8?B?UEVIM0NESDFXbGM3SDJOaGlJZ2RZNk8zenoyTXVKNTRIY3cvM3g5Z0V3Q29J?=
 =?utf-8?B?eGFkaHIrMllCemtocjZLemd4UGlHTE83bjdsMEhxTkZEcG9TSURJM3FZaDFm?=
 =?utf-8?B?d05lcDBaNEV4T0ZXZW92Y2pteTAyUGhldHFaRkNNZng0R0dsNnp3cUVMUU5t?=
 =?utf-8?B?L2c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4bca013f-c012-4dc3-7c0f-08dbd3a9ebd9
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3672.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Oct 2023 09:24:56.5406
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 3zbU1e4tHk3LFm+6LXGCgb+ixq9XrTkaqTLbENswdi7LT8j6XVj3etB+LRk0W5E2lAmyzF539HXZ3cnigWgbN0HSk8Nh98/JJlyeD4TPNcQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR11MB7719
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 23 Oct 2023 02:25:12 -0700 (PDT)

On 10/21/23 00:34, Joe Perches wrote:
> On Fri, 2023-10-20 at 15:21 +0200, Przemek Kitszel wrote:
>> Allow additional tags between Co-developed-by: and Signed-off-by:.
>> Bump severity of missing SoB to ERROR.
>>
>> Additional tags between Co-developed-by and corresponding Signed-off-by
>> could include Reviewed-by tags collected by Submitter, which is also
>> a Co-developer, but should sign-off at the very end of tags provided by
>> themself.
>>
>> Missing SoB is promoted to error while that piece of code is touched.
>>
>> Two sets of perl %hashes introduced to keep both (int) line numbers and
>> (string) messages handy for warning reporting, while keeping it correct
>> across 100+ line long commit messages.
>>
>> Mateusz Polchlopek <mateusz.polchlopek@intel.com> has reported this to me.
>>
>> Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
>> Signed-off-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
> 
> Unless this is accepted by various process folk,
> and the documentation for it updated, I think this
> should not be applied.

I will post v2 with docs updated. Would make it clear in commit message
that immediateness of SoB after CdB was important for humans checking
presence of both manually, and checkpatch has adopted such requirement
for it's own comfort.