Re: [PATCH v3 0/3] Maintainer Entry Profiles

From: Jonathan Corbet
Date: Mon Nov 25 2019 - 10:50:57 EST


On Sun, 24 Nov 2019 12:59:42 -0800
Dan Williams <dan.j.williams@xxxxxxxxx> wrote:

> Changes since v2 [1]:
> - Drop any consideration for coding style concerns in the profile. It
> was a minor aspect of the proposal that generated the bulk of the
> feedback on v2. Lets make progress on the rest.
>
> - Clarify that the "Submit Checklist Addendum" can also include details
> that submitters need to take into account before even beginning to
> craft a patch. This is in response to the RISC-V use case of
> declaring specification readiness as a patch gate, and is now also used
> by the libnvdimm subsystem to clarify details about ACPI NVDIMM Device
> Specific Method specifications. (Paul)
>
> - Non-change from v2: Kees had asked for a common directory for all
> profiles to live, but Mauro noted that this could be handled later
> with some scripting to post-process the MAINTAINERS file, or otherwise
> converting MAINTAINERS to ReST.
>
> - Clarify the cover letter to focus on the contributor focused
> Maintainer Entry Profiles, and defer discussion of a maintainer
> focused Handbook.

OK, some notes...

I wish you'd done this against docs-next, that would have saved me
resolving a few conflicts on the MAINTAINERS file.

I thought we'd agreed to move this to the process book? That said, I now
wonder about that...today I read the document as information for
maintainers on how to create their profile, so its location in the
maintainers manual is appropriate.

There were a number RST issues and warnings that I fixed up with the
following add-on patch; it was mostly a matter of adding blank lines where
needed.

One other warning resulted from the nvdimm profile document not being
linked into the TOC tree. I've added a TOC section to the new document to
bring these things together for now. This is almost certainly not what we
want in the longer term.

It was tempting to ask for this stuff to be fixed up, but I didn't want to
delay this work any longer. So it's applied to docs-next; unless something
explodes in the very near future, I intend to push this for 5.5.

Thanks,

jon