git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Satyam Sharma <satyam@infradead.org>
To: Rene Herman <rene.herman@gmail.com>
Cc: Al Viro <viro@ftp.linux.org.uk>,
	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>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Kernel Mailing List <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 19:03:53 +0530 (IST)	[thread overview]
Message-ID: <alpine.LFD.0.999.0708151846130.16414@enigma.security.iitk.ac.in> (raw)
In-Reply-To: <46C2548D.80605@gmail.com>

Hi Rene,


On Wed, 15 Aug 2007, Rene Herman wrote:

> 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

Considering some people may want to differentiate between "those who want
to be Cc'ed for patches on subsystem X" and "those who are maintainer(s)
of subsystem X", I think another "P=" kind of tag might also be useful
here.

> 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.

I'd really _love_ a tool that does all that what you've proposed above!

But why does it have to be "git-info" or anything in the git(7) suite for
that matter? This sounds like a job for a different specialised tool,
along with ".metatags" kind of files dispersed in the source tree.


Satyam

  reply	other threads:[~2007-08-15 13:21 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
2007-08-15 13:33                           ` Satyam Sharma [this message]
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=alpine.LFD.0.999.0708151846130.16414@enigma.security.iitk.ac.in \
    --to=satyam@infradead.org \
    --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=rene.herman@gmail.com \
    --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).