[PATCH] MAINTAINERS: handle buildsystem

[PATCH] MAINTAINERS: handle buildsystem

[Olaf Hering]

Tweaking configure and friends is most likely not of much interest for
anyone beside the tools maintainers.

Add a new section with files which are currently covered by "THE REST".
Also update list of stubdom related files.

Signed-off-by: Olaf Hering <olaf@aepfle.de>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Jan Beulich <jbeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>
Cc: Tim Deegan <tim@xen.org>
---

Please adjust the "M:" list as required if this patch gets applied.

 MAINTAINERS | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bf6b099..a6f4d35 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -138,6 +138,26 @@ F:	xen/drivers/char/omap-uart.c
 F:	xen/drivers/char/pl011.c
 F:	xen/drivers/passthrough/arm/
 
+BUILD SYSTEM
+M:	Ian Campbell <ian.campbell@citrix.com>
+M:	Stefano Stabellini <stefano.stabellini@citrix.com>
+M:	Ian Jackson <ian.jackson@eu.citrix.com>
+S:	Supported
+L:	xen-devel@lists.xen.org
+F:	.gitignore
+F:	config/Docs.mk.in
+F:	config/Paths.mk.in
+F:	config/Stubdom.mk.in
+F:	config/Tools.mk.in
+F:	config/Toplevel.mk.in
+F:	m4/
+F:	configure
+F:	configure.ac
+F:	stubdom/configure
+F:	stubdom/configure.ac
+F:	tools/configure
+F:	tools/configure.ac
+
 CPU POOLS
 M:	Juergen Gross <jgross@suse.com>
 S:	Supported
@@ -228,6 +248,7 @@ MINI-OS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/MiniOS.mk
 F:	extras/mini-os/
 
 OCAML TOOLS
@@ -277,6 +298,8 @@ STUB DOMAINS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/Stubdom.mk.in
+F:	m4/stubdom.m4
 F:	stubdom/
 
 TOOLSTACK

Re: [PATCH] MAINTAINERS: handle buildsystem

[Jan Beulich]

>>> On 25.09.14 at 10:59, <olaf@aepfle.de> wrote:
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -138,6 +138,26 @@ F:	xen/drivers/char/omap-uart.c
>  F:	xen/drivers/char/pl011.c
>  F:	xen/drivers/passthrough/arm/
>  
> +BUILD SYSTEM
> +M:	Ian Campbell <ian.campbell@citrix.com>
> +M:	Stefano Stabellini <stefano.stabellini@citrix.com>
> +M:	Ian Jackson <ian.jackson@eu.citrix.com>
> +S:	Supported
> +L:	xen-devel@lists.xen.org 
> +F:	.gitignore
> +F:	config/Docs.mk.in
> +F:	config/Paths.mk.in
> +F:	config/Stubdom.mk.in
> +F:	config/Tools.mk.in
> +F:	config/Toplevel.mk.in
> +F:	m4/
> +F:	configure
> +F:	configure.ac
> +F:	stubdom/configure
> +F:	stubdom/configure.ac
> +F:	tools/configure
> +F:	tools/configure.ac

With this set of files the title should be something like
CONFIGURY, as otherwise people may imply (or be tempted to
add) other makefiles here.

Jan

Re: [PATCH] MAINTAINERS: handle buildsystem

[Ian Campbell]

On Thu, 2014-09-25 at 10:59 +0200, Olaf Hering wrote:
> Tweaking configure and friends is most likely not of much interest for
> anyone beside the tools maintainers.
> 
> Add a new section with files which are currently covered by "THE REST".
> Also update list of stubdom related files.

Does anyone have an opinion on adding these new paths to the existing
tools section, rather than adding a new section?

Re: [PATCH] MAINTAINERS: handle buildsystem

[Jan Beulich]

>>> On 25.09.14 at 11:47, <Ian.Campbell@citrix.com> wrote:
> On Thu, 2014-09-25 at 10:59 +0200, Olaf Hering wrote:
>> Tweaking configure and friends is most likely not of much interest for
>> anyone beside the tools maintainers.
>> 
>> Add a new section with files which are currently covered by "THE REST".
>> Also update list of stubdom related files.
> 
> Does anyone have an opinion on adding these new paths to the existing
> tools section, rather than adding a new section?

In fact I was considering this too...

Jan

Re: [PATCH] MAINTAINERS: handle buildsystem

[Olaf Hering]

On Thu, Sep 25, Ian Campbell wrote:

> Does anyone have an opinion on adding these new paths to the existing
> tools section, rather than adding a new section?

tools is fine with me. Patch may look like shown below.

Olaf

diff --git a/MAINTAINERS b/MAINTAINERS
index bf6b099..6b10cad 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -228,6 +228,7 @@ MINI-OS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/MiniOS.mk
 F:	extras/mini-os/
 
 OCAML TOOLS
@@ -277,6 +278,8 @@ STUB DOMAINS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/Stubdom.mk.in
+F:	m4/stubdom.m4
 F:	stubdom/
 
 TOOLSTACK
@@ -285,6 +288,20 @@ M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Ian Campbell <ian.campbell@citrix.com>
 M:	Wei Liu <wei.liu2@citrix.com>
 S:	Supported
+F:	.gitignore
+F:	config/Docs.mk.in
+F:	config/Paths.mk.in
+F:	config/Stubdom.mk.in
+F:	config/Tools.mk.in
+F:	config/Toplevel.mk.in
+F:	install.sh
+F:	m4/
+F:	configure
+F:	configure.ac
+F:	docs/configure
+F:	docs/configure.ac
+F:	stubdom/configure
+F:	stubdom/configure.ac
 F:	tools/
 
 TRANSCENDENT MEMORY (TMEM)

Re: [PATCH] MAINTAINERS: handle buildsystem

[Jan Beulich]

>>> On 25.09.14 at 12:38, <olaf@aepfle.de> wrote:
> @@ -285,6 +288,20 @@ M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>  M:	Ian Campbell <ian.campbell@citrix.com>
>  M:	Wei Liu <wei.liu2@citrix.com>
>  S:	Supported
> +F:	.gitignore

I don't think .gitignore should have any specific maintainer, but apart
from that the suggested change looks mostly okay. These

> +F:	docs/configure
> +F:	docs/configure.ac
> +F:	stubdom/configure
> +F:	stubdom/configure.ac

may be misplaced, but I'm not sure. Apart from that some wildcard
use may help keep the lists of files at reasonable sizes.

Jan

Re: [PATCH] MAINTAINERS: handle buildsystem

[Olaf Hering]

On Thu, Sep 25, Jan Beulich wrote:

> may be misplaced, but I'm not sure. Apart from that some wildcard
> use may help keep the lists of files at reasonable sizes.

Wildcards are supported, I noticed that just now.

Olaf

[PATCH] MAINTAINERS: handle buildsystem

[Olaf Hering]

Tweaking configure and friends is most likely not of much interest for
anyone beside the tools maintainers.

List such files, which are currently covered by "THE REST", in the
TOOLSTACK section. Also update list of stubdom related files.

Signed-off-by: Olaf Hering <olaf@aepfle.de>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Jan Beulich <jbeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>
Cc: Tim Deegan <tim@xen.org>
---
 MAINTAINERS | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index bf6b099..d6ba866 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -228,6 +228,7 @@ MINI-OS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/MiniOS.mk
 F:	extras/mini-os/
 
 OCAML TOOLS
@@ -277,6 +278,8 @@ STUB DOMAINS
 M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Samuel Thibault <samuel.thibault@ens-lyon.org>
 S:	Supported
+F:	config/Stubdom.mk.in
+F:	m4/stubdom.m4
 F:	stubdom/
 
 TOOLSTACK
@@ -285,6 +288,16 @@ M:	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
 M:	Ian Campbell <ian.campbell@citrix.com>
 M:	Wei Liu <wei.liu2@citrix.com>
 S:	Supported
+F:	autogen.sh
+F:	config/*.in
+F:	install.sh
+F:	m4/
+F:	configure
+F:	docs/Makefile
+F:	stubdom/Makefile
+F:	*.ac
+F:	*/configure
+F:	*/*.ac
 F:	tools/
 
 TRANSCENDENT MEMORY (TMEM)

Re: [PATCH] MAINTAINERS: handle buildsystem

[Ian Campbell]

On Thu, 2014-09-25 at 15:08 +0200, Olaf Hering wrote:
> Tweaking configure and friends is most likely not of much interest for
> anyone beside the tools maintainers.
> 
> List such files, which are currently covered by "THE REST", in the
> TOOLSTACK section. Also update list of stubdom related files.
> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

Re: [PATCH] MAINTAINERS: handle buildsystem

[Ian Campbell]

On Mon, 2014-09-29 at 13:24 +0100, Ian Campbell wrote:
> On Thu, 2014-09-25 at 15:08 +0200, Olaf Hering wrote:
> > Tweaking configure and friends is most likely not of much interest for
> > anyone beside the tools maintainers.
> > 
> > List such files, which are currently covered by "THE REST", in the
> > TOOLSTACK section. Also update list of stubdom related files.
> > 
> > Signed-off-by: Olaf Hering <olaf@aepfle.de>
> 
> Acked-by: Ian Campbell <ian.campbell@citrix.com>

Thanks, I ran this via Stefano and Wei verbally and they were ok with
it, so I have applied.

Re: [PATCH] MAINTAINERS: handle buildsystem

[Ian Jackson]

Olaf Hering writes ("[PATCH] MAINTAINERS: handle buildsystem"):
> Tweaking configure and friends is most likely not of much interest for
> anyone beside the tools maintainers.
> 
> List such files, which are currently covered by "THE REST", in the
> TOOLSTACK section. Also update list of stubdom related files.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>