lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 27 Oct 2008 16:46:45 -0700
From:	Joe Perches <joe@...ches.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
	randy.dunlap@...cle.com, pavel@...e.cz, ukleinek@...len.de
Subject: Re: [PATCH 1/8] Add scripts/get_maintainer.pl

On Mon, 2008-10-27 at 15:52 -0700, Andrew Morton wrote:
> On Thu, 23 Oct 2008 16:09:19 -0700
> Joe Perches <joe@...ches.com> wrote:
> 
> > A script to parse file pattern information in MAINTAINERS
> > and return selected information about a file or patch
> > 
> > $ scripts/get_maintainer.pl
> > usage: ./scripts/get_maintainer.pl [options] patchfile
> >        ./scripts/get_maintainer.pl [options] -f file
> > version: 0.12
> 
> Example output would be nice to see.  Hopefully it is in the
> 
> 	Joe Perches <joe@...ches.com>
> 
> form, ready to be pasted into an email client?

Yes.  It's also suitable for use with git-send-email --cc-cmd

Some examples picking a random file (drivers/net/b44.c):

Get maintainers for a single file
(this includes git -by: signatories and the script does a
sort | uniq so the maintainer is not listed first)

$ ./scripts/get_maintainer.pl -f drivers/net/b44.c
"Miguel Botón" <mboton@...il.com>
Gary Zambrano <zambrano@...adcom.com>
Jeff Garzik <jeff@...zik.org>
Jeff Garzik <jgarzik@...ox.com>
John W. Linville <linville@...driver.com>
Larry Finger <Larry.Finger@...inger.net>
Michael Buesch <mb@...sch.de>
netdev@...r.kernel.org

without -git signators

$ ./scripts/get_maintainer.pl -f --nogit drivers/net/b44.c
Gary Zambrano <zambrano@...adcom.com>
Jeff Garzik <jgarzik@...ox.com>
netdev@...r.kernel.org

or without mailing lists

$ ./scripts/get_maintainer.pl -f --nogit --no-l drivers/net/b44.c
Gary Zambrano <zambrano@...adcom.com>
Jeff Garzik <jgarzik@...ox.com>

or without the maintainers, just mailing lists

$ ./scripts/get_maintainer.pl -f --nogit --no-m drivers/net/b44.c
netdev@...r.kernel.org

or get the scm entry

$ ./scripts/get_maintainer.pl -f --nogit --no-email --scm drivers/net/b44.c
git git.kernel.org/pub/scm/linux/kernel/git/jgarzik/netdev-2.6.git

> otoh, we'll sometimes also want the
> 
> 	joe@...ches.com,akpm@...ux-foundation.org
> 
> form for use by scripts.

That's the --no-multiline option

$ ./scripts/get_maintainer.pl -f --nogit --no-multiline drivers/net/b44.c
Gary Zambrano <zambrano@...adcom.com>, Jeff Garzik <jgarzik@...ox.com>, netdev@...r.kernel.org

Another possibility is to use a patch file: (random old patch file)

$ cat pad.d
diff --git a/drivers/crypto/padlock-sha.c b/drivers/crypto/padlock-sha.c
index c666b4e..1d7c328 100644
--- a/drivers/crypto/padlock-sha.c
+++ b/drivers/crypto/padlock-sha.c
@@ -251,7 +251,7 @@ static struct crypto_alg sha256_alg = {
 
 static int __init padlock_init(void)
 {
-	int rc = -ENODEV;
+	int rc;
 
 	if (!cpu_has_phe) {
 		printk(KERN_ERR PFX "VIA PadLock Hash Engine not detected.\n");
$ ./scripts/get_maintainer.pl pad.d
Herbert Xu <herbert@...dor.apana.org.au>
Jeremy Katz <katzj@...hat.com>
Suresh Siddha <suresh.b.siddha@...el.com>
Wolfgang Walter <wolfgang.walter@...m.de>

>  MAINTAINERS | 1096 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 1094 insertions(+), 2 deletions(-)
> 
> isn't practical for me or Stephen to carry, I'm afraid.  That's
> probably the most-patched file in the kernel.  This will need to be a
> direct you->Linus thing.

I also think it has to be picked up directly by Linus.

MAINTAINERS is not just probably the most patched file.
It _is_ the most patched file.

$ git log --pretty=oneline --stat | \
  grep -vP "^[a-f0-9]{40,40}" | \
  grep -vP "changed.*insertion.*deletion" | \
  cut -f1 -d"|" | sed -e 's/ //g' | \
  sort | uniq -c | sort -nr | head -5
    733 MAINTAINERS
    718 kernel/sched.c
    440 include/linux/sched.h
    411 include/linux/libata.h
    407 include/linux/ide.h

> I think the feature is sorely needed and will prove useful.

thanks...

cheers, Joe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists