Re: [RFC] automatic CC generation for patch submission

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.*:[email protected]
  scsi_*.c:[email protected]

  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:        [email protected]
 L:     [email protected]
 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:        [email protected]
 L:     [email protected]
 W:     http://sourceforge.net/projects/e1000/
 S:     Supported
+F:     drivers/net/e1000/

 INTEL PRO/10GbE SUPPORT
 P:     Jeff Kirsher
@@ -1910,6 +1913,7 @@ M:        [email protected]
 L:     [email protected]
 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux