Re: [RFC] automatic CC generation for patch submission

[Kok, Auke]

Dan Aloni wrote:
> Hello,
>
> I'd like to present a suggestion for automatic generation of 
> carbon copy fields in the E-Mails of posted patches.
>
> Basically, instead of manually figuring out who to add to CC
> when sending a patch to LKML by looking at MAINTAINERS, a 
> script can look at '.maintainers' files spread across the
> source tree and automatically generate a proper list of CCs
> for a patch.
>
> To illustrate: If a patch affects a file under 
> drivers/net/e1000, the CC script will look at these files
>
>   drivers/net/e1000/.maintainers
>   drivers/net/.maintainers
>   drivers/.maintainers
>   .maintainers
>
> ... to gather up the mailing list addresses or an individual 
> maintainer inbox address.
>
> A posssible format for this file could be a newline-separated
> list of:
>
>   [filename wildcard]:e-mail
>
> For example, drivers/scsi/.maintainers would contain:
>
>   libiscsi.*:open-iscsi@xxxxxxxxxxxxxxxx
>   scsi_*.c:scsi-devel@xxxxxxxxxxxxxxx
>
>   etc...
>
> Or, instead (or in addition) of having a '.maintainers' file 
> each directory we can modify source files by adding parsable 
> '/* MAINTAINER: name@domain */' comments. 
>
> Some extensions to the popular E-Mail clients might be needed 
> here. Also, a bot reading LKML would automatically send links 
> about posted patches to the other mailing lists whenever 
> someone forgets to add a CC.
>
> Any comments?

an easier way to implement this is to add an extra field in the MAINTAINERS 
file, something like below. All the contact info would stay the same, closely 
where applicable and it would allow you to also specify specific files as well.

Auke

(horribly whitespace-mutilated copy+paste from thunderbird below)

---
diff --git a/MAINTAINERS b/MAINTAINERS
index 4c3277c..e55be49 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -69,6 +69,7 @@ L: Mailing list that is relevant to this area
 W: Web-page with status/info
 T: SCM tree type and location.  Type is one of: git, hg, quilt.
 S: Status, one of the following:
+F: Directory tree or Files belonging to this subsystem

        Supported:      Someone is actually paid to look after this.
        Maintained:     Someone actually looks after it.
@@ -1880,6 +1881,7 @@ M:        auke-jan.h.kok@xxxxxxxxx
 L:     e1000-devel@xxxxxxxxxxxxxxxxxxxxx
 W:     http://sourceforge.net/projects/e1000/
 S:     Supported
+F:     drivers/net/e100.c

 INTEL PRO/1000 GIGABIT ETHERNET SUPPORT
 P:     Jeb Cramer
@@ -1895,6 +1897,7 @@ M:        auke-jan.h.kok@xxxxxxxxx
 L:     e1000-devel@xxxxxxxxxxxxxxxxxxxxx
 W:     http://sourceforge.net/projects/e1000/
 S:     Supported
+F:     drivers/net/e1000/

 INTEL PRO/10GbE SUPPORT
 P:     Jeff Kirsher
@@ -1910,6 +1913,7 @@ M:        auke-jan.h.kok@xxxxxxxxx
 L:     e1000-devel@xxxxxxxxxxxxxxxxxxxxx
 W:     http://sourceforge.net/projects/e1000/
 S:     Supported
+F:     drivers/net/ixgb/

 INTEL PRO/WIRELESS 2100 NETWORK CONNECTION SUPPORT
 P:     Yi Zhu
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/