lkml.org 
[lkml]   [2021]   [Jul]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] checkpatch: Add check for common mailing list helper checks
From
Date
On Fri, 2021-07-02 at 15:37 -0700, John 'Warthog9' Hawley (VMware)
wrote:
> From: John 'Warthog9' Hawley <warthog9@eaglescrag.net>
>
> Mailing lists in an attempt to try and avoid sending certain
> administrative e-mails to the list, will check the first few lines
> (usually ~10) looking for keywords. If those key words are found it
> shunts the e-mail to the list admin contact instead of potentially
> passing it through to the list.
>
> This adds a known list of the potential things that are checked for, and
> while it may search deeper into the message (it goes till it believes
> it's into the patch section) than the mailing list software this should
> help give some warning if the wrong word is potentially added in the
> wrong place in the beginning of a patch message.
[]
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
[]
> @@ -2852,6 +2852,40 @@ sub process {
[]
> +# check if this may trip up common mailing list helpers to redirect email to the admin contact
> + if ($in_commit_log &&
> + ($line =~ /\bcancel\b/i ||
> + $line =~ /\badd me\b/i ||
> + $line =~ /\bdelete me\b/i ||
> + $line =~ /\bremove\s+me\b/i ||
> + $line =~ /\bchange\b.*\baddress\b/ ||
> + $line =~ /\bsubscribe\b/i ||
> + $line =~ /^sub\b/i ||
> + $line =~ /\bunsubscribe\b/i ||
> + $line =~ /^unsub\b/i ||
> + $line =~ /^\s*help\s*$/i ||
> + $line =~ /^\s*info\s*$/i ||
> + $line =~ /^\s*info\s+\S+\s*$/i ||
> + $line =~ /^\s*lists\s*$/i ||
> + $line =~ /^\s*which\s*$/i ||
> + $line =~ /^\s*which\s+\S+\s*$/i ||
> + $line =~ /^\s*index\s*$/i ||
> + $line =~ /^\s*index\s+\S+\s*$/i ||
> + $line =~ /^\s*who\s*$/i ||
> + $line =~ /^\s*who\s+\S+\s*$/i ||
> + $line =~ /^\s*get\s+\S+\s*$/i ||
> + $line =~ /^\s*get\s+\S+\s+\S+\s*$/i ||
> + $line =~ /^\s*approve\b/i ||
> + $line =~ /^\s*passwd\b/i ||
> + $line =~ /^\s*newinfo\b/i ||
> + $line =~ /^\s*config\b/i ||
> + $line =~ /^\s*newconfig\b/i ||
> + $line =~ /^\s*writeconfig\b/i ||
> + $line =~ /^\s*mkdigest\b/i)){

Have you checked this list against actual commits?

> + WARN("MAILING LIST HELPER",

This must use underscores for the spaces

MAILING_LIST_HELPER


\
 
 \ /
  Last update: 2021-07-03 05:28    [W:0.170 / U:0.688 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site