Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Fam Zheng
On Fri, 08/11 14:06, Paolo Bonzini wrote: > On 11/08/2017 09:54, Fam Zheng wrote: > > On Thu, 08/10 12:25, Cornelia Huck wrote: > >>> I think the UI (giving no consideration to how we might implement > >>> this!) would ideally be something like: > >>> * if anybody mails a patch which touches an "u

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Paolo Bonzini
On 11/08/2017 09:54, Fam Zheng wrote: > On Thu, 08/10 12:25, Cornelia Huck wrote: >>> I think the UI (giving no consideration to how we might implement >>> this!) would ideally be something like: >>> * if anybody mails a patch which touches an "unmaintained" file, >>>a robot should send a repl

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-11 Thread Fam Zheng
On Thu, 08/10 12:25, Cornelia Huck wrote: > > I think the UI (giving no consideration to how we might implement > > this!) would ideally be something like: > > * if anybody mails a patch which touches an "unmaintained" file, > >a robot should send a reply along the lines of "thanks for the > >

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-10 Thread Cornelia Huck
On Thu, 10 Aug 2017 10:22:54 +0100 Peter Maydell wrote: > On 10 August 2017 at 09:46, Cornelia Huck wrote: > > The root problem is "some files have no maintainers". The reasons range > > from "forgot to include the file in the pattern" (easily fixed), over > > "file is updated via a script" (the

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-10 Thread Paolo Bonzini
On 10/08/2017 11:35, Peter Maydell wrote: >> actual default for un-matched: "recent contributors" + qemu-devel@ >> >> $ ./scripts/get_maintainer.pl -f disas.c >> get_maintainer.pl: No maintainers found, printing recent contributors. >> get_maintainer.pl: Do not blindly cc: them on patches! U

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-10 Thread Peter Maydell
On 9 August 2017 at 22:30, Philippe Mathieu-Daudé wrote: > I wonder if I'm understanding correctly what the MAINTAINERS file is for and > how to use it. > > From an submitter view I feel a bit confused. I thought ./get_maintainer.pl > would give me the list of person to email the changes I did on

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-10 Thread Peter Maydell
On 10 August 2017 at 09:46, Cornelia Huck wrote: > The root problem is "some files have no maintainers". The reasons range > from "forgot to include the file in the pattern" (easily fixed), over > "file is updated via a script" (the linux-headers case), to "nobody > feels up to the task" (which is

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-10 Thread Cornelia Huck
On Wed, 9 Aug 2017 18:30:25 -0300 Philippe Mathieu-Daudé wrote: > [I added mails of other person who reply to this series, this mail is > not directly addressed to Alex] Hey, a maintainership discussion! I don't think we had one before 8) > > On 08/09/2017 11:38 AM, Alex Bennée wrote: > > Phi

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-09 Thread Philippe Mathieu-Daudé
[I added mails of other person who reply to this series, this mail is not directly addressed to Alex] On 08/09/2017 11:38 AM, Alex Bennée wrote: Philippe Mathieu-Daudé writes: [...] Do you think there should be another entry in "Block QAPI, monitor, command line"? or this file (and related i

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-09 Thread Alex Bennée
Philippe Mathieu-Daudé writes: > On 08/08/2017 12:11 PM, Alex Bennée wrote: >> Philippe Mathieu-Daudé writes: > [...]>> diff --git a/MAINTAINERS b/MAINTAINERS >>> index 8cb94af6c5..6b83dac812 100644 >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -1633,6 +1633,7 @@ Common code >>> M: Richa

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-09 Thread Philippe Mathieu-Daudé
On 08/08/2017 12:11 PM, Alex Bennée wrote: Philippe Mathieu-Daudé writes: [...]>> diff --git a/MAINTAINERS b/MAINTAINERS index 8cb94af6c5..6b83dac812 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1633,6 +1633,7 @@ Common code M: Richard Henderson S: Maintained F: tcg/ +F: disas.c Is

Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-08-08 Thread Alex Bennée
Philippe Mathieu-Daudé writes: > Signed-off-by: Philippe Mathieu-Daudé > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8cb94af6c5..6b83dac812 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1633,6 +1633,7 @@ Common code > M:

[Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry

2017-07-27 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 8cb94af6c5..6b83dac812 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1633,6 +1633,7 @@ Common code M: Richard Henderson S: Maintained F: tcg/ +F: disas.c