[PATCH v2 0/1] update MAINTAINERS file with git trailers

[Bruno Larsen <blarsen@redhat.com>]

Some private chats highlighted that the currently in use git trailers
aren't explained well enough. This patch aims to fix that by adding the
information in a more verbose way to guarantee that everyone is on the
same page about them and refine anything that might still need work.

The first version of this update didn't get much traction, so I decided
to add every active global maintainer as CC, since everyone is affected
by this and I want some consensus before comitting.

Right now there is one big unanswered question: Should we have a
specific tag to explicitly signal when a patch has been partially
approved? Eli asked for it to avoid people mechanically reading tags
from thinking that a patch has been fully approved when it was only
partial. If we agree that there should be one, what would it look like?
I suggested using Acked-By, but Simon uses it differently, and I opted
to keep his usage of it.

There is a secondary question for me, which is to add a hard requirement
for approval tags. If we add it, the sourceware repository could have a
pre-receive hook (or update hook) that automatically reject pushes if
they dont have an approval tag. This would be especially useful to
avoid accidentally pushing the wrong commit, but is more annoying for
global maintainers (need to approve their own patches) and we'd need to
see what to do about the obvious fix rule. My first answer for that is
adding a special "Approved-By: Obvious fix" or similar.

Thoughts? Comments? Suggestions?

Version 1: https://inbox.sourceware.org/gdb-patches/20230516143826.3431583-1-blarsen@redhat.com/

Bruno Larsen (1):
  [gdb]: add git trailer information on gdb/MAINTAINERS

 gdb/MAINTAINERS | 56 ++++++++++++++++++++++++++++++++++++++++++-------
 1 file changed, 48 insertions(+), 8 deletions(-)

-- 
2.40.1