From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f54.google.com (mail-wm0-f54.google.com [74.125.82.54]) by dpdk.org (Postfix) with ESMTP id 728B711DE for ; Tue, 17 Jan 2017 19:42:36 +0100 (CET) Received: by mail-wm0-f54.google.com with SMTP id f73so41117689wmf.1 for ; Tue, 17 Jan 2017 10:42:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:user-agent:in-reply-to :references:mime-version:content-transfer-encoding; bh=Xt+Kmt2M8fD1PBFwxYuzZjpRRGZjzHOcGufgYy9n4pQ=; b=bQ4240RwaUueS6UuWeVZtqDeB1PI19ktMY2z1lW9u3hGDChLE7GGsiIXlwCq5Ad/y6 lsTGHjdwjr7o54BCqFDPewffKSZ+lmpN9LHfhcj/txzqEH4F/XUEqf5hBSHD+2eQFCz4 /cX053ce8Ds4Xd5y5BTiMtSFJMJJsyGDAFLyqFyILVXgq34VI2RAQcNBTxVdshLtqqPL O2LmnTAFhMMDPPV8nad8r4tG3zQ6GAJ6d8Tw8y7fmND8FbP6h2mDt7GTxIiuTReUxL+n xfEE0TDs4k6dCvEb/kcweX3V9SD+P+RdU71Xq9NBnIevSSmpdo4mnvRe5LbQM71+NDHG JS7Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=Xt+Kmt2M8fD1PBFwxYuzZjpRRGZjzHOcGufgYy9n4pQ=; b=MPk6oGdUSjYPJ1Qa3eSA9TjjzzTwr6/W9c/eufzIz9fKvaHUdltEqxEp8dDR8VqQVZ EClc5fqtZA+m1U/uBQtucXiCty9lO1rnJpjRRZC9tBvP7eLj68v0D/ouh32pGiWRv7GT VxeEacMmHmsUjLy3/7eZdM90hql00nuIx2EcuVrUx+1kehVQZYT+ZLVkLRN4JtCQ27D/ YprYKn8dVzIuOytlSUpeqog8hgxd790gqbDyR5rP1RQana2/XphA0OckQOkNllSBusgR QN/N/HCtp74fk5vQFCRgKYTQDvf62aU2pNf9NNjMTXq6r3T7UH8130pKnZ9NYjduQvnT fEyw== X-Gm-Message-State: AIkVDXIa7SByQ8baGIWgx9SOLNLSDirTzRm0XB1MFQgaard3PaunSJxKQFAZroROulFsw4nH X-Received: by 10.223.131.99 with SMTP id 90mr13297561wrd.146.1484678556089; Tue, 17 Jan 2017 10:42:36 -0800 (PST) Received: from xps13.localnet (184.203.134.77.rev.sfr.net. [77.134.203.184]) by smtp.gmail.com with ESMTPSA id u81sm38737300wmu.10.2017.01.17.10.42.35 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 17 Jan 2017 10:42:35 -0800 (PST) From: Thomas Monjalon To: Ferruh Yigit , Yuanhan Liu Cc: dev@dpdk.org, pablo.de.lara.guarch@intel.com, bruce.richardson@intel.com Date: Tue, 17 Jan 2017 19:42:33 +0100 Message-ID: <1860075.ojoIIAvjcn@xps13> User-Agent: KMail/4.14.10 (Linux/4.5.4-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: References: <1484664872-26859-1-git-send-email-thomas.monjalon@6wind.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" 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:42:36 -0000 2017-01-17 18:15, Ferruh Yigit: > 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. That's a question for Yuanhan. My comments below: Some fixes are not candidate for the stable branch because the bug was not in the previous release. These fixes are filtered out by the script devtools/git-log-fixes.sh Some fixes are not so important and we can decide they do not fit in the stable branch. Who make this decision? Relying on this Cc tag would mean the committers decide which patch to backport. > 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? No feature should be backported. > 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? Yuanhan was suggesting that the committer can do it if an author forget. > 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? I do not see the benefit of keeping other Cc in the history. It is just a convenience for authors when using git-send-email.