linux-kernel - Re: [RFC PATCH] scripts/get_maintainer.pl: add interactive mode

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]

Message-ID: <1285049533.25988.173.camel@Joe-Laptop>
Date:	Mon, 20 Sep 2010 23:12:13 -0700
From:	Joe Perches <joe@...ches.com>
To:	Florian Mickler <florian@...kler.org>
Cc:	linux-kernel@...r.kernel.org,
	"Eric W. Biederman" <ebiederm@...ssion.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Wolfram Sang <w.sang@...gutronix.de>,
	Mark Brown <broonie@...nsource.wolfsonmicro.com>,
	Stephen Hemminger <shemminger@...tta.com>,
	Stefan Richter <stefanr@...6.in-berlin.de>,
	Greg KH <greg@...ah.com>
Subject: Re: [RFC PATCH] scripts/get_maintainer.pl: add interactive mode

On Tue, 2010-09-21 at 07:31 +0200, Florian Mickler wrote:
> On Mon, 20 Sep 2010 17:38:52 -0700
> Joe Perches <joe@...ches.com> wrote:
> > > > Because git history is now not searched by default
> > > > when there is a named maintainer, there are no
> > > > commit signers.
> > > Don't know if this is intuitive. If there is the possibility to have
> > > them shown but not selected, that would be ideal as it relieves the
> > > user from pressing extra keys while still having a sane behaviour.
> > Use a command line option: --git.
> > All command line options apply to create the initial list of
> > displayed names.
> > Or add code to set
> > 	$git-fallback = 1 if $interactive;
> Are you opposed to that second solution?

Not at all.

I'd prefer to default select all returned entries though.
Maybe add some key to deselect the non-maintainer
git added entries.

> > $ git log --since=1-year-ago | grep -i "by:.*@" | \
> >   cut -f1 -d":" | sort -i | uniq -ci | sort -rn | head -10
> >   83413     Signed-off-by
> >    6544     Acked-by
> >    2022     Reviewed-by
> >    1691     Reported-by
> >    1065     Tested-by
> >     111     Reported-and-tested-by
> >      83     Suggested-by
> >      31     Requested-by
> >      28         Signed-off-by
> >      26     Fixed-by
> That's just sad. At least there are more Reviewed-By then Reported-By. 
> I don't think this actually represents reality of the
> development process. 

I think most all reviewers don't add reviewed-by acks
and most submitters don't collect them.

> > > That should just be $$bool_ref = !$$bool_ref (and probably not a
> > > function)
> > I think it needs to be a function.
> > I want a 0 or 1, not "" or 1.
> Sorry, I didn't look at the use of that. What about  
> $bool = (1 - !!$bool) 
> then? (if $bool is always 1 or 0, you can drop the double negation in
> front of it)

<shrug> Sure.  I don't care which is used.


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