Skip to content

Commit

Permalink
Remove FSF address warning from checkpatch.
Browse files Browse the repository at this point in the history
This often bites me when I'm merging changes from mainline, and never
catches a real problem in this fork.

Change-Id: I07f4c275db2c0bccb413eeba9ec1759b216f31a9
Signed-off-by: Tim Newsome <[email protected]>
  • Loading branch information
timsifive committed Apr 15, 2021
1 parent 7420382 commit aa8d30a
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions tools/scripts/checkpatch.pl
Original file line number Diff line number Diff line change
Expand Up @@ -1631,15 +1631,16 @@ sub process {
$rpt_cleaners = 1;
}

if ($rawline =~ /\bwrite to the Free/i ||
$rawline =~ /\b59\s+Temple\s+Pl/i ||
$rawline =~ /\b51\s+Franklin\s+St/i) {
my $herevet = "$here\n" . cat_vet($rawline) . "\n";
ERROR("FSF_MAILING_ADDRESS",
"Do not include the paragraph about writing to the Free Software Foundation's mailing address " .
"from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. " .
"OpenOCD already includes a copy of the GPL.\n" . $herevet)
}
# Don't care in this branch. This always messes up when we merge changes down.
#if ($rawline =~ /\bwrite to the Free/i ||
# $rawline =~ /\b59\s+Temple\s+Pl/i ||
# $rawline =~ /\b51\s+Franklin\s+St/i) {
# my $herevet = "$here\n" . cat_vet($rawline) . "\n";
# ERROR("FSF_MAILING_ADDRESS",
# "Do not include the paragraph about writing to the Free Software Foundation's mailing address " .
# "from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. " .
# "OpenOCD already includes a copy of the GPL.\n" . $herevet)
#}

# check for Kconfig help text having a real description
# Only applies when adding the entry originally, after that we do not have
Expand Down

0 comments on commit aa8d30a

Please sign in to comment.