Commit b781655a authored by Joe Perches's avatar Joe Perches Committed by Linus Torvalds

scripts/get_maintainer.pl: add maintainers in order listed in matched section

Previous behavior was "bottom-up" in each section from the pattern "F:"
entry that matched.  Now information is entered into the various lists in
the "as entered" order for each matched section.

This also allows the F: entry to be put anywhere in a section, not just as
the last entries in the section.

And a couple of improvements:

Don't alphabetically sort before outputting the matched scm, status,
subsystem and web sections.

Ignore content after a single email address so these entries are acceptable
M:	name <address> whatever other comment

And a fix:

Make an M: entry without a name again use the name from an immediately
preceding P: line if it exists.
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 11ecf53c
...@@ -313,22 +313,22 @@ if ($email || $email_list) { ...@@ -313,22 +313,22 @@ if ($email || $email_list) {
} }
if ($scm) { if ($scm) {
@scm = sort_and_uniq(@scm); @scm = uniq(@scm);
output(@scm); output(@scm);
} }
if ($status) { if ($status) {
@status = sort_and_uniq(@status); @status = uniq(@status);
output(@status); output(@status);
} }
if ($subsystem) { if ($subsystem) {
@subsystem = sort_and_uniq(@subsystem); @subsystem = uniq(@subsystem);
output(@subsystem); output(@subsystem);
} }
if ($web) { if ($web) {
@web = sort_and_uniq(@web); @web = uniq(@web);
output(@web); output(@web);
} }
...@@ -442,7 +442,7 @@ sub parse_email { ...@@ -442,7 +442,7 @@ sub parse_email {
$address = $2; $address = $2;
} elsif ($formatted_email =~ /^\s*<(.+\@\S*)>.*$/) { } elsif ($formatted_email =~ /^\s*<(.+\@\S*)>.*$/) {
$address = $1; $address = $1;
} elsif ($formatted_email =~ /^(.+\@\S*)$/) { } elsif ($formatted_email =~ /^(.+\@\S*).*$/) {
$address = $1; $address = $1;
} }
...@@ -485,12 +485,46 @@ sub format_email { ...@@ -485,12 +485,46 @@ sub format_email {
return $formatted_email; return $formatted_email;
} }
sub add_categories { sub find_starting_index {
my ($index) = @_;
while ($index > 0) {
my $tv = $typevalue[$index];
if (!($tv =~ m/^(\C):\s*(.*)/)) {
last;
}
$index--;
}
return $index;
}
sub find_ending_index {
my ($index) = @_; my ($index) = @_;
$index = $index - 1; while ($index < @typevalue) {
while ($index >= 0) {
my $tv = $typevalue[$index]; my $tv = $typevalue[$index];
if (!($tv =~ m/^(\C):\s*(.*)/)) {
last;
}
$index++;
}
return $index;
}
sub add_categories {
my ($index) = @_;
my $i;
my $start = find_starting_index($index);
my $end = find_ending_index($index);
push(@subsystem, $typevalue[$start]);
for ($i = $start + 1; $i < $end; $i++) {
my $tv = $typevalue[$i];
if ($tv =~ m/^(\C):\s*(.*)/) { if ($tv =~ m/^(\C):\s*(.*)/) {
my $ptype = $1; my $ptype = $1;
my $pvalue = $2; my $pvalue = $2;
...@@ -513,11 +547,12 @@ sub add_categories { ...@@ -513,11 +547,12 @@ sub add_categories {
} elsif ($ptype eq "M") { } elsif ($ptype eq "M") {
my ($name, $address) = parse_email($pvalue); my ($name, $address) = parse_email($pvalue);
if ($name eq "") { if ($name eq "") {
if ($index >= 0) { if ($i > 0) {
my $tv = $typevalue[$index - 1]; my $tv = $typevalue[$i - 1];
if ($tv =~ m/^(\C):\s*(.*)/) { if ($tv =~ m/^(\C):\s*(.*)/) {
if ($1 eq "P") { if ($1 eq "P") {
$name = $2; $name = $2;
$pvalue = format_email($name, $address);
} }
} }
} }
...@@ -532,11 +567,6 @@ sub add_categories { ...@@ -532,11 +567,6 @@ sub add_categories {
} elsif ($ptype eq "S") { } elsif ($ptype eq "S") {
push(@status, $pvalue); push(@status, $pvalue);
} }
$index--;
} else {
push(@subsystem,$tv);
$index = -1;
} }
} }
} }
...@@ -559,6 +589,10 @@ sub push_email_address { ...@@ -559,6 +589,10 @@ sub push_email_address {
my ($name, $address) = parse_email($line); my ($name, $address) = parse_email($line);
if ($address eq "") {
return 0;
}
if (!$email_remove_duplicates) { if (!$email_remove_duplicates) {
push(@email_to, format_email($name, $address)); push(@email_to, format_email($name, $address));
} elsif (!email_inuse($name, $address)) { } elsif (!email_inuse($name, $address)) {
...@@ -566,6 +600,8 @@ sub push_email_address { ...@@ -566,6 +600,8 @@ sub push_email_address {
$email_hash_name{$name}++; $email_hash_name{$name}++;
$email_hash_address{$address}++; $email_hash_address{$address}++;
} }
return 1;
} }
sub push_email_addresses { sub push_email_addresses {
...@@ -581,7 +617,9 @@ sub push_email_addresses { ...@@ -581,7 +617,9 @@ sub push_email_addresses {
push_email_address($entry); push_email_address($entry);
} }
} else { } else {
warn("Invalid MAINTAINERS address: '" . $address . "'\n"); if (!push_email_address($address)) {
warn("Invalid MAINTAINERS address: '" . $address . "'\n");
}
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment