doc: remove unused maintainer role from guide

Message ID 20190717181007.19958-1-thomas@monjalon.net
State Accepted, archived
Delegated to: Thomas Monjalon
Headers show
Series
  • doc: remove unused maintainer role from guide
Related show

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Thomas Monjalon July 17, 2019, 6:10 p.m.
The backup maintainer role is not explicitly used in the file MAINTAINERS.
Listing names in a priority order is preferred and more flexible
than explicit named roles.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 doc/guides/contributing/patches.rst | 2 --
 1 file changed, 2 deletions(-)

Comments

Mcnamara, John July 30, 2019, 9:36 a.m. | #1
> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Wednesday, July 17, 2019 7:10 PM
> To: Mcnamara, John <john.mcnamara@intel.com>; Kovacevic, Marko
> <marko.kovacevic@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>;
> arybchenko@solarflare.com
> Subject: [PATCH] doc: remove unused maintainer role from guide
> 
> The backup maintainer role is not explicitly used in the file MAINTAINERS.
> Listing names in a priority order is preferred and more flexible than
> explicit named roles.
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>


Acked-by: John McNamara <john.mcnamara@intel.com>
David Marchand July 30, 2019, 9:39 a.m. | #2
On Wed, Jul 17, 2019 at 8:10 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> The backup maintainer role is not explicitly used in the file MAINTAINERS.
> Listing names in a priority order is preferred and more flexible
> than explicit named roles.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  doc/guides/contributing/patches.rst | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/doc/guides/contributing/patches.rst b/doc/guides/contributing/patches.rst
> index f37fb5557..5677f1393 100644
> --- a/doc/guides/contributing/patches.rst
> +++ b/doc/guides/contributing/patches.rst
> @@ -75,7 +75,6 @@ Trees and maintainers are listed in the ``MAINTAINERS`` file. For example::
>      Crypto Drivers
>      --------------
>      M: Some Name <some.name@email.com>
> -    B: Another Name <another.name@email.com>
>      T: git://dpdk.org/next/dpdk-next-crypto
>
>      Intel AES-NI GCM PMD
> @@ -86,7 +85,6 @@ Trees and maintainers are listed in the ``MAINTAINERS`` file. For example::
>  Where:
>
>  * ``M`` is a tree or component maintainer.
> -* ``B`` is a tree backup maintainer.
>  * ``T`` is a repository tree.
>  * ``F`` is a maintained file or directory.
>
> --
> 2.21.0
>

Reviewed-by: David Marchand <david.marchand@redhat.com>
Thomas Monjalon July 30, 2019, 9:29 p.m. | #3
> > The backup maintainer role is not explicitly used in the file MAINTAINERS.
> > Listing names in a priority order is preferred and more flexible than
> > explicit named roles.
> > 
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

Applied

Patch

diff --git a/doc/guides/contributing/patches.rst b/doc/guides/contributing/patches.rst
index f37fb5557..5677f1393 100644
--- a/doc/guides/contributing/patches.rst
+++ b/doc/guides/contributing/patches.rst
@@ -75,7 +75,6 @@  Trees and maintainers are listed in the ``MAINTAINERS`` file. For example::
     Crypto Drivers
     --------------
     M: Some Name <some.name@email.com>
-    B: Another Name <another.name@email.com>
     T: git://dpdk.org/next/dpdk-next-crypto
 
     Intel AES-NI GCM PMD
@@ -86,7 +85,6 @@  Trees and maintainers are listed in the ``MAINTAINERS`` file. For example::
 Where:
 
 * ``M`` is a tree or component maintainer.
-* ``B`` is a tree backup maintainer.
 * ``T`` is a repository tree.
 * ``F`` is a maintained file or directory.