From patchwork Mon May 28 15:23:42 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Kovacevic, Marko" X-Patchwork-Id: 40479 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 480A92B96; Mon, 28 May 2018 17:23:06 +0200 (CEST) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id B035B14EC for ; Mon, 28 May 2018 17:23:04 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 May 2018 08:23:03 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,452,1520924400"; d="scan'208";a="52910972" Received: from silpixa00399502.ir.intel.com (HELO silpixa00399502.ger.corp.intel.com) ([10.237.223.218]) by FMSMGA003.fm.intel.com with ESMTP; 28 May 2018 08:23:02 -0700 From: Marko Kovacevic To: dev@dpdk.org Cc: ferruh.yigit@intel.com, john.mcnamara@intel.com, Marko Kovacevic Date: Mon, 28 May 2018 16:23:42 +0100 Message-Id: <20180528152342.22467-1-marko.kovacevic@intel.com> X-Mailer: git-send-email 2.9.5 Subject: [dpdk-dev] [PATCH v1] doc: add guides for patch fix issues X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Added contribution guideline for adding tags when sending patches that have been raised on Bugzilla Signed-off-by: Marko Kovacevic Acked-by: Ferruh Yigit --- doc/guides/contributing/patches.rst | 48 ++++++++++++++++++++++++++----------- 1 file changed, 34 insertions(+), 14 deletions(-) diff --git a/doc/guides/contributing/patches.rst b/doc/guides/contributing/patches.rst index 7bb5710..0033fd4 100644 --- a/doc/guides/contributing/patches.rst +++ b/doc/guides/contributing/patches.rst @@ -256,26 +256,46 @@ In addition to the ``Signed-off-by:`` name the commit messages can also have tags for who reported, suggested, tested and reviewed the patch being posted. Please refer to the `Tested, Acked and Reviewed by`_ section. -Coverity Related Patch Fixes -~~~~~~~~~~~~~~~~~~~~~~~~~~~~ +Patch Fix Related Issues +~~~~~~~~~~~~~~~~~~~~~~~~ -`Coverity `_ -is a tool for static code analysis. -It is used as a cloud-based service used to scan the DPDK source code, -and alert developers of any potential defects in the source code. -When fixing an issue found by Coverity, the patch must contain a Coverity issue ID -in the body of the commit message. For example:: + `Coverity `_ + is a tool for static code analysis. + It is used as a cloud-based service used to scan the DPDK source code, + and alert developers of any potential defects in the source code. + When fixing an issue found by Coverity, the patch must contain a Coverity issue ID + in the body of the commit message. For example:: - doc: fix some parameter description + doc: fix some parameter description + + Update the docs, fixing description of some parameter. + + Coverity issue: 12345 + Fixes: abcdefgh1234 ("doc: add some parameter") + Cc: author@example.com + + Signed-off-by: Alex Smith + + + `Bugzilla `_ + is a bug- or issue-tracking system. Bug-tracking + systems allow individual or groups of developers effectively to keep track of outstanding + problems with their product. When fixing an issue raised in Bugzilla, the patch must contain + a Bugzilla issue ID in the body of the commit message. For example:: + + doc: fix some parameter description + + Update the docs, fixing description of some parameter. + + Bugzilla ID: 12345 + Fixes: abcdefgh1234 ("doc: add some parameter") + Cc: author@example.com + + Signed-off-by: Alex Smith - Update the docs, fixing description of some parameter. - Coverity issue: 12345 - Fixes: abcdefgh1234 ("doc: add some parameter") - Cc: author@example.com - Signed-off-by: Alex Smith Patch for Stable Releases ~~~~~~~~~~~~~~~~~~~~~~~~~