Re: [PATCH] checkpatch.pl: Check for the FSF mailing address

From: Josh Triplett
Date: Sun Oct 06 2013 - 17:16:54 EST


On Sun, Oct 06, 2013 at 12:01:52AM -0700, Joe Perches wrote:
> On Sat, 2013-10-05 at 23:27 -0700, Greg Kroah-Hartman wrote:
> > On Sat, Oct 05, 2013 at 11:51:48AM -0700, Joe Perches wrote:
> > > On Sat, 2013-10-05 at 11:43 -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.
> > > []
> > > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> > > []
> > > > +# Check for FSF mailing addresses.
> > > > + if ($rawline =~ /You should have received a copy/ ||
> > > There are over 9,000 of these in the tree.
> > > > + $rawline =~ /write to the Free Software/ ||
> > > Over 7,000
> > > > + $rawline =~ /59 Temple Place/ ||
> > > Over 3,500
> > > > + $rawline =~ /51 Franklin Street/) {
> > > Over 1,500
> >
> > Then we should remove them all.
> []
> > I don't want to see this get worse over time, Josh, thanks for doing
> > this checkpatch patch.
>
> What about the warranty disclaimer?
>
> This program is distributed in the hope that it will be useful,
> but WITHOUT ANY WARRANTY; without even the implied warranty of
> MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> GNU General Public License for more details.

That one doesn't cause any logistical problems the way the FSF's mailing
address does, and legal departments may be uncomfortable omitting it, so
I don't think it makes sense for checkpatch to flag it.

- Josh Triplett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/