From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 436F4A0548; Wed, 15 Jun 2022 08:20:40 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F13F140222; Wed, 15 Jun 2022 08:20:39 +0200 (CEST) Received: from wout1-smtp.messagingengine.com (wout1-smtp.messagingengine.com [64.147.123.24]) by mails.dpdk.org (Postfix) with ESMTP id B9DA240220 for ; Wed, 15 Jun 2022 08:20:38 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.west.internal (Postfix) with ESMTP id 646C5320090A; Wed, 15 Jun 2022 02:20:35 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Wed, 15 Jun 2022 02:20:35 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to; s=fm2; t=1655274034; x= 1655360434; bh=+5QqguqTx1LwjCA2qN+O2RdItKbvb1t7qGGUjKeVqSU=; b=e /M3q7XDtKpg82s7akMqyLC+o1VikBvoGB/B0QW1iLG+LG5MyZsZ2wc7+2VMgDOmS JAD6229Nv8F/5CJLWFYIYT5W8gFxv2+2+CcG6fgckT+KU/tsiUAoimSGGPHiLmIE SOmctpM4YRYFhddfPxja6LR5h9SQgLHARLg6ybR/9vbO1UuGQgvfjkugc2Mh3Eq8 xw8HTE/ISXVeg5ST1lGEyaAUBjn2lz7zuIgUFUXRP51ELi7+DovjQGuouvQqpzKM uLTZJauplqUcF9MdtuVPFJDi9cGVyM+5tlPzgEL+PxrIwgug7MoagjJ3od+ihgaK f81b5jpXVRHuKBbNATnCw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1655274034; x= 1655360434; bh=+5QqguqTx1LwjCA2qN+O2RdItKbvb1t7qGGUjKeVqSU=; b=W jcceZInlDhYJV5nVR9mXujwQx3sbBc5Gicv44u1w1y+unH5bVJUdIru2kgf8WRIn 4PGlaIZ/Rv1r3VOD3o+bycmriJv/cUn2nd+ziq1f2skxce/0wPzWOf8gvf34oy/E PzWcXzc4B60XD9UOgRzZ8k38KgG0xBGCzsIM3eR6yS55U3WJcDtENU2gsb6kzOwI c1xlknsNioR54Z+LiO7h29UvPZykh1QrvI6mizhdPJKMtIOBzhPIjG9blmuJsK5X FxTOjj1aQpCMeYLSpYmwikwbsvre/ojGAo+wgcHbjt6RO5z/NU+zDwNh8wfzFjDW HjlDJ6qc+IorZHDxqlkfQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvfedruddvtddguddttdcutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefhvfevufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhho mhgrshcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqne cuggftrfgrthhtvghrnheptdejieeifeehtdffgfdvleetueeffeehueejgfeuteeftddt ieekgfekudehtdfgnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilh hfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth X-ME-Proxy: Feedback-ID: i47234305:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 15 Jun 2022 02:20:33 -0400 (EDT) From: Thomas Monjalon To: Jakub Palider Cc: dev@dpdk.org, david.marchand@redhat.com Subject: Re: [PATCH v2] devtools: ensure proper tag sequence Date: Wed, 15 Jun 2022 08:20:31 +0200 Message-ID: <5848204.lOV4Wx5bFT@thomas> In-Reply-To: <20220613222143.1436424-1-jpalider@marvell.com> References: <20220613222143.1436424-1-jpalider@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 14/06/2022 00:21, jpalider@marvell.com: > + SEQ[0] = "Coverity issue"; > + SEQ[1] = "Bugzilla ID"; > + SEQ[2] = "Fixes"; > + SEQ[3] = "Cc"; > + SEQ[4] = "^$"; > + SEQ[5] = "Suggested-by"; > + SEQ[6] = "Reported-by"; > + SEQ[7] = "Signed-off-by"; > + latest = 0; Do you think you could check that Review, Ack and Test are added after the first Signed-off? > +Tag order > +~~~~~~~~~ > + > +There is a pattern indicating how certain tags should relate to each other. > + > +Example of proper tag sequence:: > + > + Coverity issue: > + Bugzilla ID: > + Fixes: > + Cc: > + > + Suggested-by: > + Reported-by: > + Signed-off-by: Given it is an example, you could add Reviewed-by, Acked-by and Tested-by. > + > +Between first and second tag section there is and empty line. > + > +While ``Signed-off-by:`` is an obligatory tag and must exists in each commit, > +all other tags are optional. Any tag, as long as it is in proper location > +to other adjacent tags (if present), may occur multiple times. > + > +Other tags shall be laid out in a chronological order. Yes, after the first Signed-off-by.