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 B901542359; Wed, 11 Oct 2023 04:04:58 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 38F58402AE; Wed, 11 Oct 2023 04:04:58 +0200 (CEST) Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by mails.dpdk.org (Postfix) with ESMTP id 467F3400EF for ; Wed, 11 Oct 2023 04:04:56 +0200 (CEST) Received: from dggpeml100024.china.huawei.com (unknown [172.30.72.57]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4S4wwm4HWMzVlWC; Wed, 11 Oct 2023 10:01:12 +0800 (CST) Received: from [10.67.121.161] (10.67.121.161) by dggpeml100024.china.huawei.com (7.185.36.115) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.31; Wed, 11 Oct 2023 10:04:41 +0800 Subject: Re: [PATCH] doc: remove confusing command to send patch To: Thomas Monjalon , References: <20231010162635.755975-1-thomas@monjalon.net> From: fengchengwen Message-ID: <9419eb85-fd2d-cdd2-fd1f-d7ef79c7ea8d@huawei.com> Date: Wed, 11 Oct 2023 10:04:41 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: <20231010162635.755975-1-thomas@monjalon.net> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.121.161] X-ClientProxiedBy: dggems701-chm.china.huawei.com (10.3.19.178) To dggpeml100024.china.huawei.com (7.185.36.115) X-CFilter-Loop: Reflected 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 Acked-by: Chengwen Feng On 2023/10/11 0:26, Thomas Monjalon wrote: > In the contributor guide, it was said that no need to Cc maintainers > for new additions, probably for new directories not having a maintainer. > There is no harm, and it is a good habit, to always Cc maintainers. > > Remove this case as it can mislead to not Cc maintainers when needed. > > Signed-off-by: Thomas Monjalon > --- > doc/guides/contributing/patches.rst | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/doc/guides/contributing/patches.rst b/doc/guides/contributing/patches.rst > index e286d9e6d5..d114ceba2b 100644 > --- a/doc/guides/contributing/patches.rst > +++ b/doc/guides/contributing/patches.rst > @@ -561,10 +561,6 @@ Script ``get-maintainer.sh`` can be used to select maintainers automatically:: > > git send-email --to-cmd ./devtools/get-maintainer.sh --cc dev@dpdk.org 000*.patch > > -New additions can be sent without a maintainer:: > - > - git send-email --to dev@dpdk.org 000*.patch > - > You can test the emails by sending it to yourself or with the ``--dry-run`` option. > > If the patch is in relation to a previous email thread you can add it to the same thread using the Message ID:: >