Re: [PATCH 0/2] Add maintainers to the admin guide

From: Daniel Vetter
Date: Wed Dec 07 2016 - 10:47:45 EST


On Fri, Dec 02, 2016 at 10:15:13AM -0200, Mauro Carvalho Chehab wrote:
> That's my third attempt to add the MAINTAINERS contents to the admin-guide.
>
> On the past approaches, was planning to keep the documentation
> about what's at the MAINTAINERS file inside it, but that would
> require running an external script or use some Sphinx extension.
>
> This time, I took a much simpler approach: convert the initial
> part of the MAINTAINERS file to ReST and move to a file at the
> admin-guide. So, MAINTAINERS file will now contain only the
> maintainer's database, and a single line pointing to its documentation.
>
> That should hopefully be a good compromise, as we can add its
> contents to a Sphinx file without causing too much hasle.

Yeah, this seems like a rather neat approach and hopefully keeps everyone
happy. Will probably conflict since there's some in-flight patches to
document the new B: tag a bit better. Anyway:

Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>

>
>
> Mauro Carvalho Chehab (2):
> MAINTAINERS: convert first part to ReST markup
> MAINTAINERS: add it to the admin-guide
>
> Documentation/admin-guide/index.rst | 1 +
> Documentation/admin-guide/maintainers.rst | 184 ++++++++++++++++++++++++++++++
> MAINTAINERS | 125 +-------------------
> 3 files changed, 187 insertions(+), 123 deletions(-)
> create mode 100644 Documentation/admin-guide/maintainers.rst
>
> --
> 2.9.3
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-doc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch