Re: [Linux-kernel-mentees] [PATCH] Documentation: MAINTAINERS: add information about parse-maintainers.pl

[Lukas Bulwahn <lukas.bulwahn@gmail.com>]

On Fri, Nov 20, 2020 at 11:16 AM Aditya <yashsri421@gmail.com> wrote:
>
> On 20/11/20 3:42 pm, Aditya Srivastava wrote:
> > parse-maintainers.pl is a script which can be used to automatically
> > order the contents of MAINTAINERS file in preferred order (i.e.
> > 'MRLSWQBCPTFXNK').
> >
> > But currently we don't have a documentation on it.
> >
> > Add documentation regarding the same in MAINTAINERS file.
> >
> > Signed-off-by: Aditya Srivastava <yashsri421@gmail.com>
> > ---
> > the patch applies perfectly on latest next-20201119
> >
> >  MAINTAINERS | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 5db753fa1295..7ad00af7acd4 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -136,6 +136,9 @@ Descriptions of section entries and preferred order
> >             printk, pr_info or pr_err
> >          One regex pattern per line.  Multiple K: lines acceptable.
> >
> > +This order can be generated automatically for you using
> > +``scripts/parse-maintainers.pl``.
> > +
> >  Maintainers List
> >  ----------------
> >
> >
>
> Rationale for my changes:
> I did not go into much detail of its working here.
> A bit detail on this can be to use 'scripts/parse-maintainers
> --sorted' which produces the sorted file in MAINTAINERS.new by
> default. The input and output default files can be changed using
> '--input' and '--output' options. However, here I have omitted these
> details as running with '--help' option already provides this
> information in detail.
> So just making user aware of the script should serve our purpose.
>
> What do you think?
>

It makes sense not to put that into MAINTAINERS.

Do you understand how to use ./scripts/parse-maintainers, though?

So if the warning with checkpatch is shown on a patch, what would you
need to execute to get a corrected patch?

Lukas
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees