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

From: Borislav Petkov
Date: Sun Oct 06 2013 - 09:55:22 EST


On Sun, Oct 06, 2013 at 06:44:04AM -0700, Greg Kroah-Hartman wrote:
> On Sun, Oct 06, 2013 at 10:10:33AM +0200, Borislav Petkov wrote:
> > On Sun, Oct 06, 2013 at 12:05:57AM -0700, Greg Kroah-Hartman wrote:
> > > As an individual file is not the sum of the "program", I would argue
> > > that it is not needed, but some companies like to feel better by
> > > adding it. So I really don't mind either way, as it would involve
> > > arguing with lawyers about this type of thing, and I do enough of that
> > > already that I don't go out of my way to do it more...
> >
> > What about a short sentence pointing at 'COPYING' instead?
>
> Why? What would that do?

Save us the short one or two lines "this file is ..."

> Listen, all we really need is a short one or two line "this file is
> released under the FOO license" in the file. The SPDX project has the

All I'm suggesting is that it might be even simpler to add

"See COPYING in the kernel source repository for the license of this file."

or a similar text as COPYING contains the full text of the license.
Basically a pointer to it.

This has the added advantage to your short line that it points to the
exact license text and lawyers are happy.

<snip stuff we agree about anyway>

Thanks.

--
Regards/Gruss,
Boris.

Sent from a fat crate under my desk. Formatting is fine.
--
--
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/