On 26/01/2017 14:11, Thomas Huth wrote:
> This is a port of the following commit from the Linux kernel:
>
> commit e0d975b1b439c4fef58fbc306c542c94f48bb849
> Author: Joe Perches <j...@perches.com>
> Date: Wed Dec 10 15:51:49 2014 -0800
>
> checkpatch: reduce MAINTAINERS update message frequency
>
> 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 <j...@perches.com>
> Signed-off-by: Andrew Morton <a...@linux-foundation.org>
> Signed-off-by: Linus Torvalds <torva...@linux-foundation.org>
>
> Signed-off-by: Thomas Huth <th...@redhat.com>
> ---
> scripts/checkpatch.pl | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index e1be7b3..555a5b6 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1300,6 +1300,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*$/ ||
>
Maybe leave it as a warning given this change?
Paolo
