Re: [PATCH 1/5] MAINTAINERS: Add a section on git infrastructure

[Aleksandar Markovic]

On Tuesday, November 12, 2019, Alex Bennée <address@hidden> wrote:
> Aleksandar Markovic <address@hidden> writes:
>
> > From: Aleksandar Markovic <address@hidden>
> >
> > There should be a patient person maintaining gory details of
> > git-related files, and there is no better person for that role
> > than Philippe. Alex should be the reviewer for some relations
> > with gitdm.
>
> I'm not sure about this. The .gitignore files are best updated by people
> responsible for the various parts of the tree. Once out-of-tree builds
> become standard we should be able to eliminate them all together. As far
> as .mailmap is concerned I think people are quite capable of updating it
> themselves without putting the changes through a maintainer tree.

Thank you Alex for your feedback.

People here are mainly concerned about the substance of their contribution, and don't know or don't care about .mailmap file - as evidenced by many items Philippe had to add to that file. The essence of this patch was not to force people to go to the maintainer approval, but that the maintainer ftom time to time refreshes the file, if needed.

But, OK, if you have such reservations that you mentioned, I am going to remove this patch in v2. We leave all these files listed in this patch unmaintained.

Yours,

Aleksandar

 

> >
> > Signed-off-by: Aleksandar Markovic <address@hidden>
> > ---
> >  MAINTAINERS | 17 +++++++++++++++++
> >  1 file changed, 17 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 4964fbb..be43ccb 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -2664,6 +2664,23 @@ M: Daniel P. Berrange <address@hidden>
> >  S: Odd Fixes
> >  F: scripts/git-submodule.sh
> >
> > +GIT infrastructure
> > +M: Philippe Mathieu-Daudé <address@hidden>
> > +R: Alex Bennée <address@hidden>
> > +S: Maintained
> > +F: .mailmap
> > +F: scripts/git.orderfile
> > +F: .gitignore
> > +F: tests/fp/.gitignore
> > +F: tests/fp/berkeley-softfloat-3/.gitignore
> > +F: tests/fp/berkeley-testfloat-3/.gitignore
> > +F: tests/migration/.gitignore
> > +F: tests/multiboot/.gitignore
> > +F: tests/qemu-iotests/.gitignore
> > +F: tests/tcg/.gitignore
> > +F: tests/uefi-test-tools/.gitignore
> > +F: ui/keycodemapdb/tests/.gitignore
> > +
> >  Sphinx documentation configuration and build machinery
> >  M: Peter Maydell <address@hidden>
> >  S: Maintained
>
>
> --
> Alex Bennée