From: Alan Robinson <Alan.Robinson@ts.fujitsu.com>
Date: Fri, 23 Feb 2018 13:24:56 +0000 (+0100)
Subject: get_maintainers.pl: Avoid THE_REST when files are added or removed
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=140ddf18a6ec9f3e956696816af6640bbfd5283d;p=people%2Froyger%2Fxen.git

get_maintainers.pl: Avoid THE_REST when files are added or removed

When files are added or removed /dev/null is used as a place
holder name in the patch for the absent file.  Don't try and
find a MAINTAINER for this place holder, it only ever flags
and then spams THE REST, behaviour for a real filename is
unchanged.

Signed-off-by: Alan Robinson <Alan.Robinson@ts.fujitsu.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
---

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 0ce2d367fa..3fb1ad4b69 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -444,10 +444,12 @@ foreach my $file (@ARGV) {
 	    my $patch_line = $_;
 	    if (m/^\+\+\+\s+(\S+)/ or m/^---\s+(\S+)/) {
 		my $filename = $1;
-		$filename =~ s@^[^/]*/@@;
-		$filename =~ s@\n@@;
-		$lastfile = $filename;
-		push(@files, $filename);
+		if ($1 ne "/dev/null") { #Ignore the no-file placeholder
+		    $filename =~ s@^[^/]*/@@;
+		    $filename =~ s@\n@@;
+		    $lastfile = $filename;
+		    push(@files, $filename);
+		}
 		$patch_prefix = "^[+-].*";	#Now parsing the actual patch
 	    } elsif (m/^\@\@ -(\d+),(\d+)/) {
 		if ($email_git_blame) {