linux-kernel - Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag

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: <20140602192631.GA22231@linux.vnet.ibm.com>
Date:	Mon, 2 Jun 2014 12:26:31 -0700
From:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
To:	Joe Perches <joe@...ches.com>
Cc:	josh@...htriplett.org, Steven Rostedt <rostedt@...dmis.org>,
	linux-kernel@...r.kernel.org, mingo@...nel.org,
	laijs@...fujitsu.com, dipankar@...ibm.com,
	akpm@...ux-foundation.org, mathieu.desnoyers@...icios.com,
	niv@...ibm.com, tglx@...utronix.de, peterz@...radead.org,
	dhowells@...hat.com, edumazet@...gle.com, dvhart@...ux.intel.com,
	fweisbec@...il.com, oleg@...hat.com, sbw@....edu
Subject: Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag

On Mon, Jun 02, 2014 at 12:05:17PM -0700, Joe Perches wrote:
> On Mon, 2014-06-02 at 11:55 -0700, josh@...htriplett.org wrote:
> > this should go along with a change to
> > get_maintainer.pl to add those folks to the CC list.
> 
> Something like this:

To test this, I added a comment to kernel/rcu/srcu.c, then ran
scripts/get_maintainer.pl on the resulting diffs.  Without the
below change to scripts/get_maintainer.pl, I get the following:

Lai Jiangshan <laijs@...fujitsu.com> (supporter:SLEEPABLE READ-CO...)
"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com> (supporter:SLEEPABLE READ-CO...)
Dipankar Sarma <dipankar@...ibm.com> (supporter:READ-COPY UPDATE...)
linux-kernel@...r.kernel.org (open list:SLEEPABLE READ-CO...)

With the below change, it includes Josh, as expected based on the "R:"
entry I had previously added to MAINTAINERS in my local tree:

Lai Jiangshan <laijs@...fujitsu.com> (supporter:SLEEPABLE READ-CO...)
"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com> (supporter:SLEEPABLE READ-CO...)
Josh Triplett <josh@...htriplett.org> (reviewer)
Dipankar Sarma <dipankar@...ibm.com> (supporter:READ-COPY UPDATE...)
linux-kernel@...r.kernel.org (open list:SLEEPABLE READ-CO...)

So:

Tested-by: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>

> ---
>  scripts/get_maintainer.pl | 22 +++++++++++++++++++++-
>  1 file changed, 21 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 4198788..d701627 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -21,6 +21,7 @@ my $lk_path = "./";
>  my $email = 1;
>  my $email_usename = 1;
>  my $email_maintainer = 1;
> +my $email_reviewer = 1;
>  my $email_list = 1;
>  my $email_subscriber_list = 0;
>  my $email_git_penguin_chiefs = 0;
> @@ -202,6 +203,7 @@ if (!GetOptions(
>  		'remove-duplicates!' => \$email_remove_duplicates,
>  		'mailmap!' => \$email_use_mailmap,
>  		'm!' => \$email_maintainer,
> +		'r!' => \$email_reviewer,
>  		'n!' => \$email_usename,
>  		'l!' => \$email_list,
>  		's!' => \$email_subscriber_list,
> @@ -260,7 +262,8 @@ if ($sections) {
>  }
> 
>  if ($email &&
> -    ($email_maintainer + $email_list + $email_subscriber_list +
> +    ($email_maintainer + $email_reviewer +
> +     $email_list + $email_subscriber_list +
>       $email_git + $email_git_penguin_chiefs + $email_git_blame) == 0) {
>      die "$P: Please select at least 1 email option\n";
>  }
> @@ -750,6 +753,7 @@ MAINTAINER field selection options:
>      --hg-since => hg history to use (default: $email_hg_since)
>      --interactive => display a menu (mostly useful if used with the --git option)
>      --m => include maintainer(s) if any
> +    --r => include reviewer(s) if any
>      --n => include name 'Full Name <addr\@domain.tld>'
>      --l => include list(s) if any
>      --s => include subscriber only list(s) if any
> @@ -1064,6 +1068,22 @@ sub add_categories {
>  		    my $role = get_maintainer_role($i);
>  		    push_email_addresses($pvalue, $role);
>  		}
> +	    } elsif ($ptype eq "R") {
> +		my ($name, $address) = parse_email($pvalue);
> +		if ($name eq "") {
> +		    if ($i > 0) {
> +			my $tv = $typevalue[$i - 1];
> +			if ($tv =~ m/^(\C):\s*(.*)/) {
> +			    if ($1 eq "P") {
> +				$name = $2;
> +				$pvalue = format_email($name, $address, $email_usename);
> +			    }
> +			}
> +		    }
> +		}
> +		if ($email_reviewer) {
> +		    push_email_addresses($pvalue, 'reviewer');
> +		}
>  	    } elsif ($ptype eq "T") {
>  		push(@scm, $pvalue);
>  	    } elsif ($ptype eq "W") {
> 
> 
> 

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