linux-kernel - Re: [PATCH] MAINTAINERS: List the section entries in the preferred order

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAHp75VdyrvgD9GMP-gPWyHnLJC-zRcsupEhP3H1QuSuqqtUmdw@mail.gmail.com>
Date:   Fri, 27 Mar 2020 10:40:50 +0200
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Joe Perches <joe@...ches.com>
Cc:     Andrew Morton <akpm@...ux-foundation.org>,
        Andy Shevchenko <andriy.shevchenko@...el.com>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] MAINTAINERS: List the section entries in the preferred order

On Fri, Mar 27, 2020 at 1:29 AM Joe Perches <joe@...ches.com> wrote:
>
> The MAINTAINERS file header has never shown a preferred order for the
> section entries but scripts/parse-maintainers.pl added a preferred
> order with commit 61f741645a35 ("parse-maintainers: Add section pattern
> sorting")
>
> Commit 5cdbec108fd2 ("parse-maintainers: Do not sort section content by
> default") changed the preferred order to be a bit more sensible.
>
> Update the MAINTAINERS section description block to use this preferred
> section entry ordering.
>

FWIW,
Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>
thanks!

> Add a slightly better description for the N: entry too.
>
> Signed-off-by: Joe Perches <joe@...ches.com>
> ---
>  MAINTAINERS | 35 ++++++++++++++++++-----------------
>  1 file changed, 18 insertions(+), 17 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5060aa..ae4db4a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -77,21 +77,13 @@ Tips for patch submitters
>
>  8.     Happy hacking.
>
> -Descriptions of section entries
> --------------------------------
> +Descriptions of section entries and preferred order
> +---------------------------------------------------
>
>         M: *Mail* patches to: FullName <address@...ain>
>         R: Designated *Reviewer*: FullName <address@...ain>
>            These reviewers should be CCed on patches.
>         L: *Mailing list* that is relevant to this area
> -       W: *Web-page* with status/info
> -       B: URI for where to file *bugs*. A web-page with detailed bug
> -          filing info, a direct bug tracker link, or a mailto: URI.
> -       C: URI for *chat* protocol, server and channel where developers
> -          usually hang out, for example irc://server/channel.
> -       Q: *Patchwork* web based patch tracking system site
> -       T: *SCM* tree type and location.
> -          Type is one of: git, hg, quilt, stgit, topgit
>         S: *Status*, one of the following:
>            Supported:   Someone is actually paid to look after this.
>            Maintained:  Someone actually looks after it.
> @@ -102,30 +94,39 @@ Descriptions of section entries
>            Obsolete:    Old code. Something tagged obsolete generally means
>                         it has been replaced by a better system and you
>                         should be using that.
> +       W: *Web-page* with status/info
> +       Q: *Patchwork* web based patch tracking system site
> +       B: URI for where to file *bugs*. A web-page with detailed bug
> +          filing info, a direct bug tracker link, or a mailto: URI.
> +       C: URI for *chat* protocol, server and channel where developers
> +          usually hang out, for example irc://server/channel.
>         P: Subsystem Profile document for more details submitting
>            patches to the given subsystem. This is either an in-tree file,
>            or a URI. See Documentation/maintainer/maintainer-entry-profile.rst
>            for details.
> +       T: *SCM* tree type and location.
> +          Type is one of: git, hg, quilt, stgit, topgit
>         F: *Files* and directories wildcard patterns.
>            A trailing slash includes all files and subdirectory files.
>            F:   drivers/net/    all files in and below drivers/net
>            F:   drivers/net/*   all files in drivers/net, but not below
>            F:   */net/*         all files in "any top level directory"/net
>            One pattern per line.  Multiple F: lines acceptable.
> +       X: *Excluded* files and directories that are NOT maintained, same
> +          rules as F:. Files exclusions are tested before file matches.
> +          Can be useful for excluding a specific subdirectory, for instance:
> +          F:   net/
> +          X:   net/ipv6/
> +          matches all files in and below net excluding net/ipv6/
>         N: Files and directories *Regex* patterns.
> -          N:   [^a-z]tegra     all files whose path contains the word tegra

> +          N:   [^a-z]tegra     all files whose path contains tegra
> +                               (not including files like integrator)



>            One pattern per line.  Multiple N: lines acceptable.
>            scripts/get_maintainer.pl has different behavior for files that
>            match F: pattern and matches of N: patterns.  By default,
>            get_maintainer will not look at git log history when an F: pattern
>            match occurs.  When an N: match occurs, git log history is used
>            to also notify the people that have git commit signatures.
> -       X: *Excluded* files and directories that are NOT maintained, same
> -          rules as F:. Files exclusions are tested before file matches.
> -          Can be useful for excluding a specific subdirectory, for instance:
> -          F:   net/
> -          X:   net/ipv6/
> -          matches all files in and below net excluding net/ipv6/
>         K: *Content regex* (perl extended) pattern match in a patch or file.
>            For instance:
>            K: of_get_profile
>
>


-- 
With Best Regards,
Andy Shevchenko