Re: [Qemu-devel] [PATCH 5/5] checkpatch: reduce MAINTAINERS update message frequency

[Thomas Huth]

On 19.04.2018 11:11, Stefan Hajnoczi wrote:
> From: Joe Perches <address@hidden>
> 
> When files are being added/moved/deleted and a patch contains an update to
> the MAINTAINERS file, assume it's to update the MAINTAINERS file correctly
> and do not emit the "does MAINTAINERS need updating?" message.
> 
> Reported by many people.
> 
> Signed-off-by: Joe Perches <address@hidden>
> Signed-off-by: Andrew Morton <address@hidden>
> Signed-off-by: Linus Torvalds <address@hidden>
> (cherry picked from e0d975b1b439c4fef58fbc306c542c94f48bb849)
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
>  scripts/checkpatch.pl | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 95ba64f3a5..1c20c683e8 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1389,6 +1389,12 @@ sub process {
>                       }
>               }
>  
> +# Check if MAINTAINERS is being updated.  If so, there's probably no need to
> +# emit the "does MAINTAINERS need updating?" message on file add/move/delete
> +             if ($line =~ /^\s*MAINTAINERS\s*\|/) {
> +                     $reported_maintainer_file = 1;
> +             }
> +
>  # Check for added, moved or deleted files
>               if (!$reported_maintainer_file && !$in_commit_log &&
>                   ($line =~ /^(?:new|deleted) file mode\s*\d+\s*$/ ||
> 

Reviewed-by: Thomas Huth <address@hidden>