From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] doc: add details on requirements for patch ack and merge Date: Tue, 21 Feb 2017 15:12:32 +0100 Message-ID: <5210332.Rcp5tEyIor@xps13> References: <1487678567-13375-1-git-send-email-bruce.richardson@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, techboard@dpdk.org To: Bruce Richardson Return-path: Received: from mail-wm0-f50.google.com (mail-wm0-f50.google.com [74.125.82.50]) by dpdk.org (Postfix) with ESMTP id F33732A5E for ; Tue, 21 Feb 2017 15:12:33 +0100 (CET) Received: by mail-wm0-f50.google.com with SMTP id v77so76520336wmv.0 for ; Tue, 21 Feb 2017 06:12:33 -0800 (PST) In-Reply-To: <1487678567-13375-1-git-send-email-bruce.richardson@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2017-02-21 12:02, Bruce Richardson: > +#. Once a patch has been acked by the relevant maintainer, reviewers may still comment on it for a further > + two weeks. After that time, the patch should be merged into the relevent git tree for the next release. > + Additional notes and restrictions: > + > + * patches should be acked by a maintainer at least two days before the release merge > + deadline, in order to make that release. > + * for patches acked with less than two weeks to go to the merge deadline, all additional > + comments should be made no later than two days before the merge deadline. Do we need a drawing to illustrate the deadlines? :) > + * after the appropriate time for additional feedback has passed, if the patch has not yet > + been merged to the relevant tree by the committer, it should be treated as though it had, > + in that any additional changes needed to it must be addressed by a follow-on patch, rather > + than rework of the original. > + * trivial patches may be merged sooner than described above at the tree committer's > + discretion. Is it a trivial patch or should I wait 2 weeks before merging this one? Acked-by: Thomas Monjalon