(Adding Andrew Morton)
On Sun, 2013-10-06 at 15:23 -0700, Josh Triplett wrote:
> Kernel maintainers reject new instances of the GPL boilerplate paragraph
> directing people to write to the FSF for a copy of the GPL, since the
> FSF has moved in the past and may do so again.
>
> Make this an error for new code, but just a --strict CHK in --file mode;
> anyone interested in doing tree-wide cleanups of this form can enable
> this test explicitly.
>
> Signed-off-by: Josh Triplett <j...@joshtriplett.org>
> ---
>
> v2: Make this a CHK in --file mode, using a code pattern suggested by
> Joe Perches.
>
> scripts/checkpatch.pl | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 2ee9eb7..9161533 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1859,6 +1859,18 @@ sub process {
> $rpt_cleaners = 1;
> }
>
> +# Check for FSF mailing addresses.
> + if ($rawline =~ /You should have received a copy/ ||
> + $rawline =~ /write to the Free Software/ ||
> + $rawline =~ /59 Temple Place/ ||
> + $rawline =~ /51 Franklin Street/) {
> + my $herevet = "$here\n" . cat_vet($rawline) . "\n";
> + my $msg_type = \&ERROR;
> + $msg_type = \&CHK if ($file);
> + &{$msg_type}("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. Linux 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
> # sufficient context to determine whether it is indeed long enough.
Acked-by: Joe Perches <j...@perches.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
