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

From: Aleksandar Markovic <amarkovic@wavecomp.com>

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.

Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com>
---
 MAINTAINERS | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Aleksandar Markovic <aleksandar.markovic@rt-rk.com> writes:

> From: Aleksandar Markovic <amarkovic@wavecomp.com>
>
> 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.

>
> Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com>
> ---
>  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 <berrange@redhat.com>
>  S: Odd Fixes
>  F: scripts/git-submodule.sh
>
> +GIT infrastructure
> +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> +R: Alex Bennée <alex.bennee@linaro.org>
> +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 <peter.maydell@linaro.org>
>  S: Maintained


--
Alex Bennée

On Tuesday, November 12, 2019, Alex Bennée <alex.bennee@linaro.org> wrote:

>
> Aleksandar Markovic <aleksandar.markovic@rt-rk.com> writes:
>
> > From: Aleksandar Markovic <amarkovic@wavecomp.com>
> >
> > 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 <amarkovic@wavecomp.com>
> > ---
> >  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 <berrange@redhat.com>
> >  S: Odd Fixes
> >  F: scripts/git-submodule.sh
> >
> > +GIT infrastructure
> > +M: Philippe Mathieu-Daudé <philmd@redhat.com>
> > +R: Alex Bennée <alex.bennee@linaro.org>
> > +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 <peter.maydell@linaro.org>
> >  S: Maintained
>
>
> --
> Alex Bennée
>
>