From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id D27C6DE3 for ; Tue, 17 Jan 2017 19:15:10 +0100 (CET) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga102.fm.intel.com with ESMTP; 17 Jan 2017 10:15:08 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,245,1477983600"; d="scan'208";a="49768502" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by orsmga004.jf.intel.com with ESMTP; 17 Jan 2017 10:15:07 -0800 To: Thomas Monjalon , dev@dpdk.org References: <1484664872-26859-1-git-send-email-thomas.monjalon@6wind.com> Cc: Yuanhan Liu From: Ferruh Yigit Message-ID: Date: Tue, 17 Jan 2017 18:15:06 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <1484664872-26859-1-git-send-email-thomas.monjalon@6wind.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH] devtools: check stable tag in fixes 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: , X-List-Received-Date: Tue, 17 Jan 2017 18:15:11 -0000 On 1/17/2017 2:54 PM, Thomas Monjalon wrote: > The tag "Cc: stable@dpdk.org" must be set when the commit must be > backported to a stable branch. > > It must be located just below the "Fixes:" tag (without blank line) > and followed by a blank line, separated from SoB and review tags below. I am OK to keep it if it will help stable tree maintenance, but I still not clear about why we need this. If a patch is a fix, it should already have "Fixes:" line, so this can be used to parse git history. If patch is a feature, as far as I know still can go to stable tree, but for this case stable tree maintainer decides this, and author putting "Cc: stable@dpdk.org" tag not so useful. Author can put this tag just for recommendation, but if so why we are saving this into git history? Initially this was to be sure fixes CC'ed to stable mail list, now meaning is changing I guess. For the case author already cc'ed the stable tree but not put Cc: tag into git commit, should committer add this explicitly or ask from author a new version of the patch? Last thing, if this tag will remain in the commit log, is this only for stable tree, or any "Cc: " can stay in the history? > > Signed-off-by: Thomas Monjalon > --- <...>