git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rene Herman <rene.herman@gmail.com>
To: Al Viro <viro@ftp.linux.org.uk>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Joe Perches <joe@perches.com>,
	git@vger.kernel.org, Junio C Hamano <gitster@pobox.com>,
	Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Arjan van de Ven <arjan@infradead.org>,
	Trond Myklebust <trond.myklebust@fys.uio.no>,
	Mariusz Kozlowski <m.kozlowski@tuxland.pl>,
	akpm@linux-foundation.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] [1/2many] - FInd the maintainer(s) for a patch - scripts/get_maintainer.pl
Date: Wed, 15 Aug 2007 03:19:09 +0200	[thread overview]
Message-ID: <46C2548D.80605@gmail.com> (raw)
In-Reply-To: <20070814193333.GI21089@ftp.linux.org.uk>

On 08/14/2007 09:33 PM, Al Viro wrote:

> FWIW, I suspect that we are looking at that from the wrong POV.  If
> that's about "who ought to be Cc'd on the issues dealing with <list
> of pathnames>", why does it have to be tied to "who is maintainer for
> <pathname>"?
> 
> I'm not suggesting something like fs.ext2@kernel.org with something
> like majordomo allowing to add yourself to those, but something less
> extreme in that direction might be worth thinking about...  Hell,
> even simple
> $ finger fs/minix/dir.c@cc.kernel.org
> with majordomo-like interface for adding yourself to such lists
> might solve most of those problems...

It mostly is just about that it seems. However, this would not also allow 
the other information currently in the MAINTAINERS file to be queried in 
similar ways.

Git could grow a generic file meta data implementation through the use of 
tags, sort of like tags on multimedia files although while with multimedia 
files the tags are in fact stored as a file header, here you'd keep them 
just in git. Any project using git would be free to define its own set of 
info tags and you'd supply them to git simply as a list of

<tag>=<value>

pairs:

$ git info --add drivers/ide/ide-cd.c <<EOF
CC="Alan Cox <alan@lxorguk.ukuu.org.uk>", linux-ide@vger.kernel.org
EOF

Or as a more expansive example, with the tags set on a directory (and the 
output shown this time):

$ git info drivers/infiniband/
CC="Roland Dreier <rolandd@cisco.com>"
CC="Sean Hefty <mshefty@ichips.intel.com>"
CC="Hal Rosenstock <halr@voltaire.com>"
CC=openib-general@openib.org
W=http://www.openib.org/
T=git kernel.org:/pub/scm/linux/kernel/git/roland/infiniband.git

$ git info --type="W" drivers/infiniband/
http://www.openib.org/

The project can link the actual tags such as CC, W and T to --options for 
the "info" command in the git configuration file for the tree (and/or just 
define a few upfront I guess) making it look nicer:

$ git info --cc drivers/infiniband/
"Roland Dreier <rolandd@cisco.com>"
"Sean Hefty <mshefty@ichips.intel.com>"
"Hal Rosenstock <halr@voltaire.com>"
openib-general@openib.org

$ git info --website drivers/infiniband/
http://www.openib.org/

$ git info --tree drivers/infiniband/
git kernel.org:/pub/scm/linux/kernel/git/roland/infiniband.git

Extra: when you have such an implementation, you can use it for other 
purposes as well such as the summary Documentation/ files want for the 
00-INDEX files:

$ git info --summary Documentation/BUG-HUNTING
brute force method of doing binary search of patches to find bug.

And importantly -- when queuried for a file that itself doesn't have the 
requested info tag:

$ git info --cc drivers/infiniband/core/addr.c

git looks for the tag on the drivers/infiniband/core/ directory next, and 
then on drivers/infiniband/, where it finds it. linux-kernel@vger.kernel.org 
would be the final fallback, being set on the project root.

I'd really like something like this. As long as projects are both free to 
use and not use them and free to define their own set of tags I believe this 
would work very nicely.

Once you have these tags, you can basically use them for anything.

Rene.

  parent reply	other threads:[~2007-08-15  1:46 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1186984174.10249.7.camel@localhost>
     [not found] ` <200708131933.10125.m.kozlowski@tuxland.pl>
     [not found]   ` <1187026955.2688.4.camel@laptopd505.fenrus.org>
     [not found]     ` <1187037445.6628.98.camel@heimdal.trondhjem.org>
     [not found]       ` <1187054366.2757.0.camel@laptopd505.fenrus.org>
     [not found]         ` <46C10AA8.3090505@gmail.com>
     [not found]           ` <20070814102033.604c8695@the-village.bc.nu>
     [not found]             ` <46C1CFFE.4000001@gmail.com>
2007-08-14 17:00               ` [PATCH] [1/2many] - FInd the maintainer(s) for a patch - scripts/get_maintainer.pl Joe Perches
2007-08-14 18:03                 ` Rene Herman
2007-08-14 18:28                   ` Joe Perches
2007-08-14 18:33                     ` Rene Herman
2007-08-14 18:40                     ` Linus Torvalds
2007-08-14 18:54                       ` Joe Perches
2007-08-14 19:33                       ` Al Viro
2007-08-14 19:57                         ` Joe Perches
2007-08-15  1:19                         ` Rene Herman [this message]
2007-08-15 13:33                           ` Satyam Sharma
2007-08-15 13:39                             ` Rene Herman
2007-08-15 13:52                               ` Kyle Moffett
2007-08-16 10:58                                 ` Rene Herman
2007-08-16 11:08                                   ` Rene Herman
2007-08-16 11:26                                     ` Salikh Zakirov
2007-08-16 11:57                                       ` Rene Herman
2007-08-16 15:40                                   ` Al Viro
2007-08-16 15:53                                     ` Rene Herman
2007-08-16 19:00                                   ` Junio C Hamano
2007-08-17  4:24                                     ` Rene Herman
2007-08-15 19:37                         ` Krzysztof Halasa
2007-08-15 23:19                           ` Al Viro
2007-08-15  1:35                       ` Richard Knutsson
2007-08-15  9:29                       ` Stefan Richter
2007-08-15 15:31                         ` Ray Lee
2007-08-16 20:36                       ` Joe Perches
2007-08-15  1:31                 ` Junio C Hamano
2007-08-15  2:12                   ` Joe Perches
2007-08-15  5:25                     ` Junio C Hamano
2007-08-15  5:42                       ` Rene Herman
2007-08-15  9:39                     ` Stefan Richter
2007-08-15 11:44                       ` Rene Herman
2007-08-15 17:26                         ` Joe Perches
2007-08-17  2:13                   ` Joe Perches
2007-08-17  2:30                     ` Joe Perches
2007-08-17 17:54                       ` [PATCH] - git-send-email.perl Joe Perches
2007-08-17 23:38                         ` Junio C Hamano
2007-08-18  1:51                           ` Joe Perches

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=46C2548D.80605@gmail.com \
    --to=rene.herman@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=arjan@infradead.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=m.kozlowski@tuxland.pl \
    --cc=torvalds@linux-foundation.org \
    --cc=trond.myklebust@fys.uio.no \
    --cc=viro@ftp.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).